Skip to content

Comments

made install command mac-compatible#62

Merged
PetteriAimonen merged 1 commit intoPetteriAimonen:masterfrom
j-krose:fix-install
May 11, 2025
Merged

made install command mac-compatible#62
PetteriAimonen merged 1 commit intoPetteriAimonen:masterfrom
j-krose:fix-install

Conversation

@j-krose
Copy link
Contributor

@j-krose j-krose commented May 10, 2025

When running make install on MacOS I was get the following output:

% make install
which ronn && make update_docs || true
install -D build/focus-stack "/usr/local/bin/focus-stack"
usage: install [-bCcpSsUv] [-f flags] [-g group] [-m mode] [-o owner]
               [-M log] [-D dest] [-h hash] [-T tags]
               [-B suffix] [-l linkflags] [-N dbdir]
               file1 file2
       install [-bCcpSsUv] [-f flags] [-g group] [-m mode] [-o owner]
               [-M log] [-D dest] [-h hash] [-T tags]
               [-B suffix] [-l linkflags] [-N dbdir]
               file1 ... fileN directory
       install -dU [-vU] [-g group] [-m mode] [-N dbdir] [-o owner]
               [-M log] [-D dest] [-h hash] [-T tags]
               directory ...
make: *** [install] Error 64

man install (on MacOS) says:

     -D destdir
             Specify the DESTDIR (top of the file hierarchy) that the items are installed in to.  If -M
             metalog is in use, a leading string of “destdir” will be removed from the file names logged
             to the metalog.  This option does not affect where the actual files are installed.

which is different than the functionality on linux:

-D
    create all leading components of DEST except the last, or all components of --target-directory, then copy SOURCE to DEST

To avoid this incompatibility I just replaced the -D with a mkdir -p to emulate the same behavior.

After this change, running make install works properly; focus-stack is on my path and man focus-stack shows the proper man page.

@j-krose
Copy link
Contributor Author

j-krose commented May 10, 2025

Addresses #63

@j-krose
Copy link
Contributor Author

j-krose commented May 10, 2025

@PetteriAimonen FYI

@PetteriAimonen PetteriAimonen merged commit 43949ff into PetteriAimonen:master May 11, 2025
1 check failed
@PetteriAimonen
Copy link
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants