Clean up improper dependencies on gettext without LOCALE
Get series with b4
b4 am 20121014210734.17845.64519.stgit@localhostVersions newest first
- v1 Latest10 patches 0 repliesNew
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/10 | diffutils: only needs gettext if locale is selectedpackage/ package/ | +2 | 1 reply | |
| 2/10 | flex: only needs gettext if locale is selectedpackage/ package/ | +2 | 1 reply | |
| 3/10 | ndisc6: only needs gettext if locale is selectedpackage/ package/ | +2 | 1 reply | |
| 4/10 | util-linux: only needs gettext if locale is selectedpackage/ package/ | +2 | 1 reply | |
| 5/10 | lshw: fix build with no LOCALE supportpackage/ package/ | +10 | 0 replies | |
| 6/10 | avahi: remove double dependency on gettextpackage/ | +1 | 1 reply | |
| 7/10 | pulseaudio: remove dependency on gettextpackage/ package/ | +0 | 1 reply | |
| 8/10 | gdk-pixbuf: remove dependency on gettextpackage/ package/ | +0 | 0 replies | |
| 9/10 | glib-networking: remove redundant dependency on gettextpackage/ | +1 | 1 reply | |
| 9/10 | glib-networking: remove redundant dependency on gettext | +0 | 0 replies | |
| 9/10 | glib-networking: remove redundant dependency on gettext | +0 | 0 replies | |
| 10/10 | libsoup: remove redundant dependency on gettextpackage/ | +1 | 1 reply | |
| 10/10 | libsoup: remove redundant dependency on gettext | +0 | 0 replies |
Reviews trailers per patch
5 of 13 patches have a Reviewed-by or Acked-by.
| Patch | Yann E. MORIN |
|---|---|
| 1/10 | A-b |
| 2/10 | A-b |
| 3/10 | A-b |
| 4/10 | A-b |
| 5/10 | |
| 6/10 | A-b |
| 7/10 | |
| 8/10 | |
| 9/10 | |
| 9/10 | |
| 9/10 | |
| 10/10 | |
| 10/10 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Buildroot] [PATCH 00/10] Clean up improper dependencies on gettext without LOCALE
This is a resend of a series that cleans up redundant dependencies on gettext, in packages that select gettext when LOCALE is not enabled.
The first 7 patches really remove the gettext dependency. The last three are in fact a NOP, because libglib2 selects gettext unconditionally. I have kept the NEEDS_GETTEXT_IF_LOCALE dependency for these packages, because they do have i18n code and do require libintl. But the dependency adds nothing, because libglib2 does the same already.
There are no changes compared to the first send, except for patch 7 which received a comment from Thomas Petazzoni.
Regards,
Arnout
---
Arnout Vandecappelle (Essensium/Mind) (10):
diffutils: only needs gettext if locale is selected
flex: only needs gettext if locale is selected
ndisc6: only needs gettext if locale is selected
util-linux: only needs gettext if locale is selected
lshw: fix build with no LOCALE support
avahi: remove double dependency on gettext
pulseaudio: remove dependency on gettext
gdk-pixbuf: remove dependency on gettext
glib-networking: remove redundant dependency on gettext
libsoup: remove redundant dependency on gettext
package/avahi/avahi.mk | 5 +---- package/diffutils/Config.in | 1 + package/diffutils/diffutils.mk | 3 +-- package/flex/Config.in | 1 + package/flex/flex.mk | 3 +-- package/gdk-pixbuf/Config.in | 1 - package/gdk-pixbuf/gdk-pixbuf.mk | 1 - package/glib-networking/Config.in | 2 +- package/libsoup/Config.in | 2 +- package/lshw/Config.in | 2 +- package/lshw/lshw.mk | 12 +++++++++--- package/multimedia/pulseaudio/Config.in | 1 - package/multimedia/pulseaudio/pulseaudio.mk | 1 - package/ndisc6/Config.in | 1 + package/ndisc6/ndisc6.mk | 2 +- package/util-linux/Config.in | 1 + package/util-linux/util-linux.mk | 2 +- 17 files changed, 21 insertions(+), 20 deletions(-)