Makefile fixes #325

2013-07-23T08:31:36+02:00

postmodern commented

2013-07-23 08:31:36 +02:00

(Migrated from github.com)

Fixed a typo in uninstall where a .svg wasn't being `rm -f'ed.
Install into /usr/local by default, but allow the PREFIX variable to be overridden.

- Fixed a typo in `uninstall` where a `.svg` wasn't being `rm -f'ed. - Install into `/usr/local` by default, but allow the `PREFIX` variable to be overridden.

ghost commented

2013-07-23 11:29:24 +02:00

(Migrated from github.com)

Is installing to /usr/local by default recommended?

postmodern commented

2013-07-23 11:53:00 +02:00

(Migrated from github.com)

/usr is reserved for software installed by the package manager, /usr/local is reserved for software installed by the system administrator. So your rpm/deb/etc specs should continue to install into /usr, but make install should definitely install into /usr/local. See the FHS for more info.

`/usr` is reserved for software installed by the package manager, `/usr/local` is reserved for software installed by the system administrator. So your rpm/deb/etc specs should continue to install into `/usr`, but `make install` should definitely install into `/usr/local`. See the [FHS](http://www.pathname.com/fhs/pub/fhs-2.3.html#THEUSRHIERARCHY) for more info.

ghost commented

2013-07-23 12:20:09 +02:00

(Migrated from github.com)

Since debian/rules and RPM spec just call make install how can the system tell the difference between an installation from a package and a manual make install?

postmodern commented

2013-07-23 12:44:16 +02:00

(Migrated from github.com)

For debian/rules I have seen this done:

override_dh_auto_test-indep:

override_dh_auto_build-indep:

%:
  PREFIX=/usr dh $@

I would consult a second opinion on debian packaging etiquette, since another developer submitted the above code to one of my projects.

For rpm/*.spec I do this:

%install
make install PREFIX=%{buildroot}/usr

For `debian/rules` I have seen this done: ``` override_dh_auto_test-indep: override_dh_auto_build-indep: %: PREFIX=/usr dh $@ ``` I would consult a second opinion on debian packaging etiquette, since another developer [submitted the above code to one of my projects](https://github.com/postmodern/chruby/pull/169). For `rpm/*.spec` I do this: ``` %install make install PREFIX=%{buildroot}/usr ```

This repo is archived. You cannot comment on pull requests.

No reviewers