WolframRhodium / VapourSynth-dpid

Rapid, Detail-Preserving Image Downscaler for VapourSynth
BSD 3-Clause "New" or "Revised" License
33 stars 4 forks source link

missing step when install (create the target directory) #6

Closed sl1pkn07 closed 5 years ago

sl1pkn07 commented 5 years ago

https://github.com/WolframRhodium/VapourSynth-dpid/blob/7771a30a9c9dea222261311718d3b82be4f05a83/Source/Makefile#L28-L29

seems need a intermediary step for create the target

cp: cannot create regular file '/tmp/makepkg/vapoursynth-plugin-dpid-git/pkg/vapoursynth-plugin-dpid-git/usr/lib/vapoursynth': No such file or directory

instead copy and chmod, better use install -Dm644 this command do the same

diff --git a/Source/Makefile b/Source/Makefile
index b670ac2..b18c520 100644
--- a/Source/Makefile
+++ b/Source/Makefile
@@ -25,8 +25,7 @@ kernel16.o: kernel16.cu

 install: $(TARGET)
    @echo "Installing..."
-   @cp $(TARGET) $(PREFIX)
-   @chmod 644 $(PREFIX)/$(TARGET)
+   @install -Dm644 $(TARGET) $(PREFIX)/$(TARGET)

 uninstall:
    @echo "Uninstalling..."

greetings

fdar0536 commented 5 years ago

VapourSynth-dpid/Source/Makefile

Lines 28 to 29 in 7771a30

@cp $(TARGET) $(PREFIX) @chmod 644 $(PREFIX)/$(TARGET) seems need a intermediary step for create the target

cp: cannot create regular file '/tmp/makepkg/vapoursynth-plugin-dpid-git/pkg/vapoursynth-plugin-dpid-git/usr/lib/vapoursynth': No such file or directory

instead copy and chmod, better use install -Dm644 this command do the same

diff --git a/Source/Makefile b/Source/Makefile
index b670ac2..b18c520 100644
--- a/Source/Makefile
+++ b/Source/Makefile
@@ -25,8 +25,7 @@ kernel16.o: kernel16.cu

 install: $(TARGET)
  @echo "Installing..."
- @cp $(TARGET) $(PREFIX)
- @chmod 644 $(PREFIX)/$(TARGET)
+ @install -Dm644 $(TARGET) $(PREFIX)/$(TARGET)

 uninstall:
  @echo "Uninstalling..."

greetings

Sorry, I forgot that step.

WolframRhodium commented 5 years ago

Thanks you guys.