Message ID | 20121014211110.17845.3033.stgit@localhost |
---|---|
State | Superseded |
Headers | show |
Arnout, All, On Sunday 14 October 2012 Arnout Vandecappelle (Essensium/Mind) wrote: > Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> > --- > package/glib-networking/Config.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/package/glib-networking/Config.in b/package/glib-networking/Config.in > index dab35ea..f1aa7ee 100644 > --- a/package/glib-networking/Config.in > +++ b/package/glib-networking/Config.in > @@ -2,7 +2,7 @@ config BR2_PACKAGE_GLIB_NETWORKING > bool "glib-networking" > depends on BR2_USE_WCHAR # glib2 > select BR2_PACKAGE_LIBGLIB2 > - select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT > + select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT_IF_LOCALE > help > Network-related GIO modules for glib. The commit message says "remove redundant dependency on gettext", but this change changes the dependency on plain gettext to gettext-if-locales. Thus, I think the subject is wrong, and should be changed to something like: glib-networking: only depends on gettext if locales are used Regards, Yann E. MORIN.
On 10/14/12 23:21, Yann E. MORIN wrote: > Arnout, All, > > On Sunday 14 October 2012 Arnout Vandecappelle (Essensium/Mind) wrote: >> Signed-off-by: Arnout Vandecappelle (Essensium/Mind)<arnout@mind.be> >> --- >> package/glib-networking/Config.in | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/package/glib-networking/Config.in b/package/glib-networking/Config.in >> index dab35ea..f1aa7ee 100644 >> --- a/package/glib-networking/Config.in >> +++ b/package/glib-networking/Config.in >> @@ -2,7 +2,7 @@ config BR2_PACKAGE_GLIB_NETWORKING >> bool "glib-networking" >> depends on BR2_USE_WCHAR # glib2 >> select BR2_PACKAGE_LIBGLIB2 >> - select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT >> + select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT_IF_LOCALE >> help >> Network-related GIO modules for glib. > > The commit message says "remove redundant dependency on gettext", but this > change changes the dependency on plain gettext to gettext-if-locales. > > Thus, I think the subject is wrong, and should be changed to something like: > glib-networking: only depends on gettext if locales are used Actually, on second revision: glib-networking uses gettext directly so we should therefore select it here as well. It worked fine in testing because libglib2 anyway depends on locale. Therefore, I'll replace this with unconditionally depending on gettext in the .mk file. Regards, Arnout
On 11/01/12 14:48, Arnout Vandecappelle wrote: > On 10/14/12 23:21, Yann E. MORIN wrote: >> Arnout, All, >> >> On Sunday 14 October 2012 Arnout Vandecappelle (Essensium/Mind) wrote: >>> Signed-off-by: Arnout Vandecappelle (Essensium/Mind)<arnout@mind.be> >>> --- >>> package/glib-networking/Config.in | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/package/glib-networking/Config.in b/package/glib-networking/Config.in >>> index dab35ea..f1aa7ee 100644 >>> --- a/package/glib-networking/Config.in >>> +++ b/package/glib-networking/Config.in >>> @@ -2,7 +2,7 @@ config BR2_PACKAGE_GLIB_NETWORKING >>> bool "glib-networking" >>> depends on BR2_USE_WCHAR # glib2 >>> select BR2_PACKAGE_LIBGLIB2 >>> - select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT >>> + select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT_IF_LOCALE >>> help >>> Network-related GIO modules for glib. >> >> The commit message says "remove redundant dependency on gettext", but this >> change changes the dependency on plain gettext to gettext-if-locales. >> >> Thus, I think the subject is wrong, and should be changed to something like: >> glib-networking: only depends on gettext if locales are used > > Actually, on second revision: glib-networking uses gettext directly so we should > therefore select it here as well. It worked fine in testing because libglib2 > anyway depends on locale. > > Therefore, I'll replace this with unconditionally depending on gettext in > the .mk file. ... but of course, the whole point of this series was to remove BR2_NEEDS_GETTEXT... I'm just going to drop this patch for the time being, which leaves us with 3 packages using BR2_NEEDS_GETTEXT: php (gettext module), libglib2, glib-networking. Regards, Arnout
diff --git a/package/glib-networking/Config.in b/package/glib-networking/Config.in index dab35ea..f1aa7ee 100644 --- a/package/glib-networking/Config.in +++ b/package/glib-networking/Config.in @@ -2,7 +2,7 @@ config BR2_PACKAGE_GLIB_NETWORKING bool "glib-networking" depends on BR2_USE_WCHAR # glib2 select BR2_PACKAGE_LIBGLIB2 - select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT + select BR2_PACKAGE_GETTEXT if BR2_NEEDS_GETTEXT_IF_LOCALE help Network-related GIO modules for glib.
Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> --- package/glib-networking/Config.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)