Message ID | 20161230191557.76657402C1F77@oldenburg.str.redhat.com |
---|---|
State | New |
Headers | show |
On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: > This change also removes the preprocessor-based function renaming. > It also applied to tests in resolv/, which ended up running against > the historic functions. This is missing a NEWS entry. Andreas.
On 02/16/2017 01:38 AM, Andreas Schwab wrote: > On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: > >> This change also removes the preprocessor-based function renaming. >> It also applied to tests in resolv/, which ended up running against >> the historic functions. > > This is missing a NEWS entry. How so? I truly wasn't aware that the change is user-visible. Or put differently, I don't think we need NEWS entries if we turn symbols which are not mentioned in any header file into compat symbols because there is no supported ways programs can link to these symbols. Thanks, Florian
On Feb 16 2017, Florian Weimer <fweimer@redhat.com> wrote: > On 02/16/2017 01:38 AM, Andreas Schwab wrote: >> On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: >> >>> This change also removes the preprocessor-based function renaming. >>> It also applied to tests in resolv/, which ended up running against >>> the historic functions. >> >> This is missing a NEWS entry. > > How so? I truly wasn't aware that the change is user-visible. ekiga fails to build. https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:B:DVD/ekiga/standard/x86_64 Andreas.
* Andreas Schwab: > On Feb 16 2017, Florian Weimer <fweimer@redhat.com> wrote: > >> On 02/16/2017 01:38 AM, Andreas Schwab wrote: >>> On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: >>> >>>> This change also removes the preprocessor-based function renaming. >>>> It also applied to tests in resolv/, which ended up running against >>>> the historic functions. >>> >>> This is missing a NEWS entry. >> >> How so? I truly wasn't aware that the change is user-visible. > > ekiga fails to build. > > https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:B:DVD/ekiga/standard/x86_64 This seems to be a bogus configure check, probably a leftover from the libc5 days. I don't the source code references to the res_gethostbyaddr function itself. ypbind is another candidate for problems, but it's intimately tied to glibc anyway and is thus a special case. I still don't think this warrants a NEWS entry.
On 02/16/2017 07:32 AM, Florian Weimer wrote: > * Andreas Schwab: > >> On Feb 16 2017, Florian Weimer <fweimer@redhat.com> wrote: >> >>> On 02/16/2017 01:38 AM, Andreas Schwab wrote: >>>> On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: >>>> >>>>> This change also removes the preprocessor-based function renaming. >>>>> It also applied to tests in resolv/, which ended up running against >>>>> the historic functions. >>>> >>>> This is missing a NEWS entry. >>> >>> How so? I truly wasn't aware that the change is user-visible. >> >> ekiga fails to build. >> >> https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:B:DVD/ekiga/standard/x86_64 > > This seems to be a bogus configure check, probably a leftover from the > libc5 days. I don't the source code references to the > res_gethostbyaddr function itself. > > ypbind is another candidate for problems, but it's intimately tied to > glibc anyway and is thus a special case. > > I still don't think this warrants a NEWS entry. I agree, but it does need a distribution note in the 2.26 release wiki. https://sourceware.org/glibc/wiki/Release/2.26#Packaging_Changes
* Carlos O'Donell: > On 02/16/2017 07:32 AM, Florian Weimer wrote: >> * Andreas Schwab: >> >>> On Feb 16 2017, Florian Weimer <fweimer@redhat.com> wrote: >>> >>>> On 02/16/2017 01:38 AM, Andreas Schwab wrote: >>>>> On Dez 30 2016, fweimer@redhat.com (Florian Weimer) wrote: >>>>> >>>>>> This change also removes the preprocessor-based function renaming. >>>>>> It also applied to tests in resolv/, which ended up running against >>>>>> the historic functions. >>>>> >>>>> This is missing a NEWS entry. >>>> >>>> How so? I truly wasn't aware that the change is user-visible. >>> >>> ekiga fails to build. >>> >>> https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:B:DVD/ekiga/standard/x86_64 >> >> This seems to be a bogus configure check, probably a leftover from the >> libc5 days. I don't the source code references to the >> res_gethostbyaddr function itself. >> >> ypbind is another candidate for problems, but it's intimately tied to >> glibc anyway and is thus a special case. >> >> I still don't think this warrants a NEWS entry. > > I agree, but it does need a distribution note in the 2.26 release wiki. > https://sourceware.org/glibc/wiki/Release/2.26#Packaging_Changes Right, I added something to the end of: https://sourceware.org/glibc/wiki/Release/2.25
diff --git a/include/resolv.h b/include/resolv.h index 699cc81..95dcd3c 100644 --- a/include/resolv.h +++ b/include/resolv.h @@ -26,7 +26,6 @@ extern __thread struct __res_state *__resp attribute_tls_model_ie; extern int __res_vinit (res_state, int); extern int __res_maybe_init (res_state, int); extern void _sethtent (int); -extern void _endhtent (void); extern struct hostent *_gethtent (void); extern struct hostent *_gethtbyname (const char *__name); extern struct hostent *_gethtbyname2 (const char *__name, int __af); diff --git a/resolv/Makefile b/resolv/Makefile index 06329e1..bd086e0 100644 --- a/resolv/Makefile +++ b/resolv/Makefile @@ -42,11 +42,12 @@ routines += gai_sigqueue tests += tst-res_hconf_reorder endif extra-libs-others = $(extra-libs) -libresolv-routines := gethnamaddr res_comp res_debug \ +libresolv-routines := res_comp res_debug \ res_data res_mkquery res_query res_send \ inet_net_ntop inet_net_pton inet_neta base64 \ ns_parse ns_name ns_netint ns_ttl ns_print \ - ns_samedomain ns_date compat-hooks + ns_samedomain ns_date \ + compat-hooks compat-gethnamaddr libanl-routines := gai_cancel gai_error gai_misc gai_notify gai_suspend \ getaddrinfo_a @@ -84,12 +85,6 @@ generated += mtrace-tst-leaks.out tst-leaks.mtrace \ include ../Rules -CPPFLAGS += -Dgethostbyname=res_gethostbyname \ - -Dgethostbyname2=res_gethostbyname2 \ - -Dgethostbyaddr=res_gethostbyaddr \ - -Dgetnetbyname=res_getnetbyname \ - -Dgetnetbyaddr=res_getnetbyaddr - CFLAGS-res_hconf.c = -fexceptions # The DNS NSS modules needs the resolver. diff --git a/resolv/gethnamaddr.c b/resolv/compat-gethnamaddr.c similarity index 91% rename from resolv/gethnamaddr.c rename to resolv/compat-gethnamaddr.c