Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 226839 - x11-libs/libxklavier: patch (libxklavier-1.14-werror.patch) changes both autotools source and result
Summary: x11-libs/libxklavier: patch (libxklavier-1.14-werror.patch) changes both auto...
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: bad-autotools
  Show dependency tree
 
Reported: 2008-06-14 17:43 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2008-06-21 22:30 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Diego Elio Pettenò (RETIRED) gentoo-dev 2008-06-14 17:43:33 UTC
The patch in summary is touching both Makefile.in and Makefile.am, probably to avoid autotools rebuild.

This treatment is usually reserved for a few selected system packages that cannot have their autotool scripts rebuilt.

This _could_ cause maintainermode-driven rebuild (see http://blog.flameeyes.eu/articles/2008/06/13/maintaner-mode ), which is something we should be avoiding as much as possible.

Please just patch Makefile.am and/or configure.in/.ac and rebuild autotools, unless you have very good reasons not to.
Comment 1 Mart Raudsepp gentoo-dev 2008-06-21 22:30:18 UTC
I'm sorry, but there is no technical reason this could cause any problems, other than getting the emerge succeed too fast thanks to the lack of an eautoreconf call.
Feel free to open with a technical reason for not patching Makefile.in's, and
therefore forcing to use eautoreconf and a longer emerge time on users.
Meanwhile, INVALID.