bartobri / no-more-secrets

A command line tool that recreates the famous data decryption effect seen in the 1992 movie Sneakers.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

"make install" target broken on macOS in 0.3.1

ilovezfs opened this issue · comments

==> make prefix=/usr/local/Cellar/no-more-secrets/0.3.1 install
install -d /usr/local/Cellar/no-more-secrets/0.3.1/share/man/man6
install -m644 -t /usr/local/Cellar/no-more-secrets/0.3.1/share/man/man6 nms.6 sneakers.6
install: illegal option -- t
usage: install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
               [-o owner] file1 file2
       install [-bCcpSsv] [-B suffix] [-f flags] [-g group] [-m mode]
               [-o owner] file1 ... fileN directory
       install -d [-v] [-g group] [-m mode] [-o owner] directory ...
make: *** [install] Error 64

This was not broken in 0.3.0.

Build log here: https://bot.brew.sh/job/Homebrew%20Core/16712/version=sierra/consoleText

@susnux Can we install the man pages without the use of the -t option? This is causing an issue on OSX.

Of course, if you do not mind I create a pull request fixing this.

@ilovezfs Can you confirm this is fixed? Do you need me to create another release?

@bartobri yes, but I've discovered a new bug now :)

iMac-TMP:~ joe$ brew test no-more-secrets
Testing no-more-secrets
==> Using the sandbox
==> /usr/local/Cellar/no-more-secrets/0.3.1/bin/nms -v
Error: no-more-secrets: failed
<"nms version 0.3.1"> expected but was
<"nms version 0.3.0">.

Do you need me to create another release?

A new release also with the matching version number would be great.

Woops! That's not the first time I've forgotten to increment the software version number to match a new release. Sorry about that.

I've created a new release (0.3.2) with matching version number. Please confirm all is working as expected.

Looks like we're good: Homebrew/homebrew-core#10142

Great! Thanks!

Thanks for the quick turnaround! New version shipped.