Skip to content
This repository was archived by the owner on Jul 19, 2018. It is now read-only.

add make install target for homebrew formula#6

Open
kouk wants to merge 3 commits into
tiwe-de:masterfrom
kouk:makeinstall
Open

add make install target for homebrew formula#6
kouk wants to merge 3 commits into
tiwe-de:masterfrom
kouk:makeinstall

Conversation

@kouk

@kouk kouk commented Oct 9, 2016

Copy link
Copy Markdown

Homebrew maintainers asked for a make install target. Would be great if you could merge this.

@tiwe-de

tiwe-de commented Oct 9, 2016

Copy link
Copy Markdown
Owner

This only installs some of the files and doesn't respect the DESTDIR variable.

@kouk

kouk commented Oct 9, 2016

Copy link
Copy Markdown
Author

@tiwe-de I added DESTDIR support.

@MikeMcQuaid

Copy link
Copy Markdown

This only installs some of the files

Which files are missing?

This is why I suggested we add a make install for Homebrew otherwise it's a bit unclear what files should be installed.

@tiwe-de tiwe-de self-assigned this Oct 10, 2016
@tiwe-de

tiwe-de commented Oct 10, 2016

Copy link
Copy Markdown
Owner

I commented on this already in the homebrew pull request.

An install target would depend on all and install basically anything the Debian package has. @kouk's install target misses afssh.1, ssh-askpass-noinput + manpage and the bash-completion functions.

I'll work on that these days.

@tiwe-de

tiwe-de commented Oct 10, 2016

Copy link
Copy Markdown
Owner

@MikeMcQuaid how are the paths you use for bash-completion files, READMEs and changelogs?

@MikeMcQuaid

Copy link
Copy Markdown

@tiwe-de We use standard paths for all so feel free to make make install put everything in the Debian-usual locations.

Comment thread Makefile
CXXFLAGS ?= -g -O2 -Wall -Wold-style-cast
PREFIX ?= /usr/local
bindir = $(PREFIX)/bin
mandir = $(PREFIX)/man

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure about Debian but are these not usually in share/man? If not, could you make this MANDIR ?= so Homebrew could customise it? Thanks!

@tiwe-de

tiwe-de commented Oct 11, 2016

Copy link
Copy Markdown
Owner

Unfortunately adding an install: all target makes the Debian package build it twice, the second time under fakeroot. I'll have to find a solution which makes the version.h target less phony, first.

@kouk

kouk commented Oct 23, 2016

Copy link
Copy Markdown
Author

@tiwe-de if you think I can contribute some way let me know. I'm a bit confused as to what the status is.

@kouk

kouk commented Nov 12, 2016

Copy link
Copy Markdown
Author

@tiwe-de any ideas about this? Do you think you will find time to implement it? If not I can attempt to do it according to the comments above.

@tiwe-de

tiwe-de commented Nov 13, 2016

Copy link
Copy Markdown
Owner

I need a solution where:

  • version.h is regenerated when the checked-out version in git changes
  • version.h is not touched when the source is not in git (extracted from a tarball)
  • nothing is rebuilt when make is run twice in a row

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants