Move <rpc/netdb.h> from sunrpc to inet
Commit Message
Restore <rpc/netdb.h> as an installed header. Delete the dummy header
resolv/rpc/netdb.h because inet is not an optional glibc component
(so its <rpc/netdb.h> is always available).
Fixes commit acb527929d0c2b3bb0798472c42ddb3203729708 ("Move
non-deprecated RPC-related functions from sunrpc to inet") in
combination with commit 5500cdba4018ddbda7909bc7f4f9718610b43cf0
("Remove --enable-obsolete-rpc configure flag").
Tested on i686-linux-gnu. Manually verified that
/usr/include/rpc/netdb.h is installed.
---
include/rpc/netdb.h | 2 +-
inet/Makefile | 3 ++-
{sunrpc => inet}/rpc/netdb.h | 0
resolv/rpc/netdb.h | 3 ---
4 files changed, 3 insertions(+), 5 deletions(-)
Comments
On Jul 14 2020, Florian Weimer wrote:
> Restore <rpc/netdb.h> as an installed header. Delete the dummy header
> resolv/rpc/netdb.h because inet is not an optional glibc component
> (so its <rpc/netdb.h> is always available).
This is ok. Though, I wonder wheather we actually need rpc/netdb.h,
instead of putting its contents into netdb.h.
There is also a stale comment in resolv/netdb.h.
Andreas.
* Andreas Schwab:
> On Jul 14 2020, Florian Weimer wrote:
>
>> Restore <rpc/netdb.h> as an installed header. Delete the dummy header
>> resolv/rpc/netdb.h because inet is not an optional glibc component
>> (so its <rpc/netdb.h> is always available).
>
> This is ok. Though, I wonder wheather we actually need rpc/netdb.h,
> instead of putting its contents into netdb.h.
Yes, we should probably deprecate that header.
> There is also a stale comment in resolv/netdb.h.
Which comment is stale? This one?
/* This is necessary to make this include file properly replace the
Sun version. */
It's old, but probably still technically correct?
Thanks,
Florian
The 07/14/2020 12:30, Florian Weimer via Libc-alpha wrote:
> Restore <rpc/netdb.h> as an installed header. Delete the dummy header
> resolv/rpc/netdb.h because inet is not an optional glibc component
> (so its <rpc/netdb.h> is always available).
>
> Fixes commit acb527929d0c2b3bb0798472c42ddb3203729708 ("Move
> non-deprecated RPC-related functions from sunrpc to inet") in
> combination with commit 5500cdba4018ddbda7909bc7f4f9718610b43cf0
> ("Remove --enable-obsolete-rpc configure flag").
>
> Tested on i686-linux-gnu. Manually verified that
> /usr/include/rpc/netdb.h is installed.
what's the status of this?
we have some build breakage without this.
it looks OK to me.
* Szabolcs Nagy:
> The 07/14/2020 12:30, Florian Weimer via Libc-alpha wrote:
>> Restore <rpc/netdb.h> as an installed header. Delete the dummy header
>> resolv/rpc/netdb.h because inet is not an optional glibc component
>> (so its <rpc/netdb.h> is always available).
>>
>> Fixes commit acb527929d0c2b3bb0798472c42ddb3203729708 ("Move
>> non-deprecated RPC-related functions from sunrpc to inet") in
>> combination with commit 5500cdba4018ddbda7909bc7f4f9718610b43cf0
>> ("Remove --enable-obsolete-rpc configure flag").
>>
>> Tested on i686-linux-gnu. Manually verified that
>> /usr/include/rpc/netdb.h is installed.
>
> what's the status of this?
>
> we have some build breakage without this.
>
> it looks OK to me.
It's blocked, pending release manager approval.
Thanks,
Florian
On 7/17/20 3:56 AM, Florian Weimer wrote:
> * Szabolcs Nagy:
>
>> The 07/14/2020 12:30, Florian Weimer via Libc-alpha wrote:
>>> Restore <rpc/netdb.h> as an installed header. Delete the dummy header
>>> resolv/rpc/netdb.h because inet is not an optional glibc component
>>> (so its <rpc/netdb.h> is always available).
>>>
>>> Fixes commit acb527929d0c2b3bb0798472c42ddb3203729708 ("Move
>>> non-deprecated RPC-related functions from sunrpc to inet") in
>>> combination with commit 5500cdba4018ddbda7909bc7f4f9718610b43cf0
>>> ("Remove --enable-obsolete-rpc configure flag").
>>>
>>> Tested on i686-linux-gnu. Manually verified that
>>> /usr/include/rpc/netdb.h is installed.
>>
>> what's the status of this?
>>
>> we have some build breakage without this.
>>
>> it looks OK to me.
>
> It's blocked, pending release manager approval.
Approved for 2.32. Please push with fixed resolv/Makefile stale
reference fixed.
@@ -1,5 +1,5 @@
#ifndef _RPC_NETDB_H
-#include <sunrpc/rpc/netdb.h>
+#include <inet/rpc/netdb.h>
# ifndef _ISOMAC
@@ -25,7 +25,8 @@ include ../Makeconfig
headers := netinet/ether.h netinet/in.h netinet/in_systm.h \
netinet/if_ether.h netinet/igmp.h \
netinet/tcp.h netinet/ip.h $(wildcard arpa/*.h protocols/*.h) \
- aliases.h ifaddrs.h netinet/ip6.h netinet/icmp6.h bits/in.h
+ aliases.h ifaddrs.h netinet/ip6.h netinet/icmp6.h bits/in.h \
+ rpc/netdb.h
routines := htonl htons \
inet_lnaof inet_mkadr \
similarity index 100%
rename from sunrpc/rpc/netdb.h
rename to inet/rpc/netdb.h
deleted file mode 100644
@@ -1,3 +0,0 @@
-/* This is a dummy file for <rpc/netdb.h>, which is included by <netdb.h>.
- This file is installed when the C library does not support the SunRPC
- interfaces (including 'struct rpcent' et al) at all. */