standardml / smackage

Smackage Package Manager for Standard ML
Other
122 stars 12 forks source link

Ensure $(DESTDIR)/bin exists before installing smackage binary #25

Closed pzel closed 8 years ago

pzel commented 8 years ago

Trying to install smackage with a nonexistent DESTDIR via make install produces the following error:

$ DESTDIR=/home/p/.smackage make install
rm -f /home/p/.smackage/bin/smackage.new
cp bin/smackage /home/p/.smackage/bin/smackage.new
cp: cannot create regular file ‘/home/p/.smackage/bin/smackage.new’: No such file or directory
make: *** [install] Error 1

This patch makes sure that the required directory exists before attempting to install files.

gian commented 8 years ago

Cool, seems very reasonable. Thanks!

gian commented 8 years ago

@pzel I'll merge this and bump to 1.4.1

pzel commented 8 years ago

Glad to have helped!