Details at
https://fedoraproject.org/wiki/Make_install_packaging_draft
It is just a very minor enhancement.
Counter proposal: Just enhance this section:
http://fedoraproject.org/wiki/Packaging:Guidelines#Why_the_.25makeinstall_macro_should_not_be_used
To mention %{make_install} in addition to make install DESTDIR=%{?_buildroot}. Request rpmdev-newspec to use %{make_install} in the templates it generates.
(I say this because we don't actually currently tell people to use make install DESTDIR[...]... except in the context of telling them not to use %makeinstall.)
If someone could also generate a list of all places in current guidelines that we use make install DESTDIR[...], we could change those to use the macros as well.
Either way would work but I think we should avoid introducing double negation in some indirect way but adding a recommendation within a note not to use something. Rewriting that section would make more sense IMO. Updating rpmdev-newspec is a good suggestion. I have filed https://bugzilla.redhat.com/show_bug.cgi?id=828455. Thanks.
As I won't be able to attend this week's FPC meeting, here is my vote: -1
Rationale: I don't see %make_install to provide any advantage over make install DESTDIR=..., esp. when considering that make install DESTDIR=... is well known and understood for many, many years.
That said, to me, all %make_install does, is to add further macro-bloat and incompatibilities to older Fedora releases and to other distros.
I am strongly in favor of intentionally not recommend using %make_install, but to use make install DESTDIR=... everywhere.
@toshio, http://git.fedorahosted.org/git/?p=rpmdevtools.git;a=commitdiff;h=9f4736665d84ee683565532fa52a31e695252d87 implements your suggestion.
abadger1999's suggestion to amend the language to "Instead, Fedora packages should use: %make_install (Note the "_"!), make DESTDIR=%{buildroot} install or make DESTDIR=$RPM_BUILD_ROOT install. Those all do the same thing." is approved (+1:6, 0:0, -1:1)
text has been added to the Guidelines.
Announcement text:
Mention that the %make_install macro (not to be confused with %makeinstall) may be used instead of "make DESTDIR=%{buildroot}".
Metadata Update from @toshio: - Issue assigned to spot
Login to comment on this ticket.