Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Petr Vorel <petr.vorel@gmail.com>
To: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Cc: buildroot@buildroot.org
Subject: [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch
Date: Sun, 25 Jul 2021 22:14:53 +0200	[thread overview]
Message-ID: <YP3GPccSpJbFMAgv@pevik> (raw)

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

             reply	other threads:[~2021-07-25 20:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-25 20:14 Petr Vorel [this message]
2021-07-25 21:14 ` [Buildroot] [RFC] rpcbind: Remove 0001-Remove-yellow-pages-support.patch Thomas Petazzoni
2021-07-25 21:20   ` Petr Vorel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=YP3GPccSpJbFMAgv@pevik \
    --to=petr.vorel@gmail.com \
    --cc=buildroot@buildroot.org \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox