* [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch
@ 2021-07-25 20:14 Petr Vorel
2021-07-25 21:14 ` Thomas Petazzoni
0 siblings, 1 reply; 3+ messages in thread
From: Petr Vorel @ 2021-07-25 20:14 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: buildroot
Hi Thomas,
I'm thinking about removing
package/rpcbind/0001-Remove-yellow-pages-support.patch patch.
It was originally added in 2012 in 8bc245da21 ("rpcbind: new package").
At the time constants YPBINDPROC_SETDOM, YPPROG, YPPASSWDPROG were missing in
rpcbind (added in 2015 in rpcbind commit 9e0c7ac ("configure.ac: Check for rpcsvc/mount.h").
I'll verify it, but I'll strongly suspect the only problem was in these missing
constants and very old libtirpc. Thus it should be working now.
Probably the best approach would be to disable legacy yellow pages in upstream,
I can ask for some configure switch. But if you're don't require it, I'd just
drop the patch which is not needed any more.
Patch was also somehow related to package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch
(originally added as package/libtirpc/libtirpc-0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch),
which was removed in 883a33ece5 ("libtirpc: remove NIS patch") as not needed.
Kind regards,
Petr
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch
2021-07-25 20:14 [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch Petr Vorel
@ 2021-07-25 21:14 ` Thomas Petazzoni
2021-07-25 21:20 ` Petr Vorel
0 siblings, 1 reply; 3+ messages in thread
From: Thomas Petazzoni @ 2021-07-25 21:14 UTC (permalink / raw)
To: Petr Vorel; +Cc: buildroot
Hello Petr,
On Sun, 25 Jul 2021 22:14:53 +0200
Petr Vorel <petr.vorel@gmail.com> wrote:
> I'm thinking about removing
> package/rpcbind/0001-Remove-yellow-pages-support.patch patch.
>
> It was originally added in 2012 in 8bc245da21 ("rpcbind: new package").
> At the time constants YPBINDPROC_SETDOM, YPPROG, YPPASSWDPROG were missing in
> rpcbind (added in 2015 in rpcbind commit 9e0c7ac ("configure.ac: Check for rpcsvc/mount.h").
>
> I'll verify it, but I'll strongly suspect the only problem was in these missing
> constants and very old libtirpc. Thus it should be working now.
> Probably the best approach would be to disable legacy yellow pages in upstream,
> I can ask for some configure switch. But if you're don't require it, I'd just
> drop the patch which is not needed any more.
>
> Patch was also somehow related to package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch
> (originally added as package/libtirpc/libtirpc-0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch),
> which was removed in 883a33ece5 ("libtirpc: remove NIS patch") as not needed.
Sure, if it all builds/works fine with a more recent libtirpc, I'm all
fine with dropping this patch!
Best regards,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch
2021-07-25 21:14 ` Thomas Petazzoni
@ 2021-07-25 21:20 ` Petr Vorel
0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2021-07-25 21:20 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: buildroot
Hi Thomas,
> Hello Petr,
> On Sun, 25 Jul 2021 22:14:53 +0200
> Petr Vorel <petr.vorel@gmail.com> wrote:
> > I'm thinking about removing
> > package/rpcbind/0001-Remove-yellow-pages-support.patch patch.
> > It was originally added in 2012 in 8bc245da21 ("rpcbind: new package").
> > At the time constants YPBINDPROC_SETDOM, YPPROG, YPPASSWDPROG were missing in
> > rpcbind (added in 2015 in rpcbind commit 9e0c7ac ("configure.ac: Check for rpcsvc/mount.h").
> > I'll verify it, but I'll strongly suspect the only problem was in these missing
> > constants and very old libtirpc. Thus it should be working now.
> > Probably the best approach would be to disable legacy yellow pages in upstream,
> > I can ask for some configure switch. But if you're don't require it, I'd just
> > drop the patch which is not needed any more.
> > Patch was also somehow related to package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch
> > (originally added as package/libtirpc/libtirpc-0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch),
> > which was removed in 883a33ece5 ("libtirpc: remove NIS patch") as not needed.
> Sure, if it all builds/works fine with a more recent libtirpc, I'm all
> fine with dropping this patch!
Thanks! I'll verify it and send patch or report if still needed.
Kind regards,
Petr
> Best regards,
> Thomas
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-25 21:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-07-25 20:14 [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch Petr Vorel
2021-07-25 21:14 ` Thomas Petazzoni
2021-07-25 21:20 ` Petr Vorel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox