diff mbox

[1/1] libtirpc : always include svc_auth_none

Message ID 1402779467-13505-1-git-send-email-sagaert.johan@skynet.be
State Superseded
Headers show

Commit Message

Sagaert Johan June 14, 2014, 8:57 p.m. UTC
This function is needed in rpcbind >= 0.2.1
without this patch the linking of rpcbind > 0.2.1 will fail.
(based on http://www.spinics.net/lists/linux-nfs/msg27277.html )

Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
---
 ...btirpc-0011-always-include-svc_auth_none.patch | 39 ++++++++++++++++++++++
 1 file changed, 39 insertions(+)
 create mode 100644 package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch

Comments

Yann E. MORIN June 14, 2014, 9:04 p.m. UTC | #1
Sagaert, All,

On 2014-06-14 22:57 +0200, Sagaert Johan spake thusly:
> This function is needed in rpcbind >= 0.2.1
> without this patch the linking of rpcbind > 0.2.1 will fail.
> (based on http://www.spinics.net/lists/linux-nfs/msg27277.html )

Does that fix an autobuilder issue? If so, please add:

    Fixes:
        http://autobuild.b.n/results/XXX/SHA1

> Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
> ---
>  ...btirpc-0011-always-include-svc_auth_none.patch | 39 ++++++++++++++++++++++
>  1 file changed, 39 insertions(+)
>  create mode 100644 package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> 
> diff --git a/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> new file mode 100644
> index 0000000..66c660c
> --- /dev/null
> +++ b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> @@ -0,0 +1,39 @@
> +From 02c9f3898e4189bb0ec817d492b36da6de31abf8 Mon Sep 17 00:00:00 2001
> +From: Sagaert Johan <sagaert.johan@skynet.be>
> +Date: Sat, 14 Jun 2014 22:02:59 +0200
> +Subject: [PATCH 1/1] always include svc_auth_none
> +
> +(needed by rpcbind >=0.2.1 )
> +
> +Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>

Since you use the patch as-is, you should keep the authorship of the
patch, and just add your own SoB line.

Regards,
Yann E. MORIN.

> +---
> + src/Makefile.am | 7 +++----
> + 1 file changed, 3 insertions(+), 4 deletions(-)
> +
> +diff --git a/src/Makefile.am b/src/Makefile.am
> +index bd276cf..a6313b7 100644
> +--- a/src/Makefile.am
> ++++ b/src/Makefile.am
> +@@ -48,16 +48,15 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln
> +         getrpcport.c mt_misc.c pmap_clnt.c pmap_getmaps.c pmap_getport.c \
> +         pmap_prot.c pmap_prot2.c pmap_rmt.c rpc_prot.c rpc_commondata.c \
> +         rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \
> +-        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_generic.c \
> +-        svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
> ++        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \
> ++        svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
> + 
> + ## XDR
> + libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c
> + 
> + ## Secure-RPC
> + if GSS
> +-    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c \
> +-		svc_auth_none.c
> ++    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c
> +     libtirpc_la_LDFLAGS += $(GSSGLUE_LIBS)
> +     libtirpc_la_CFLAGS = -DHAVE_RPCSEC_GSS $(GSSGLUE_CFLAGS)
> + endif
> +-- 
> +2.0.0
> +
> -- 
> 2.0.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Sagaert Johan June 14, 2014, 9:27 p.m. UTC | #2
-----Oorspronkelijk bericht-----
Van: Yann E. MORIN [mailto:yann.morin.1998@gmail.com] Namens Yann E. MORIN
Verzonden: zaterdag 14 juni 2014 23:04
Aan: Sagaert Johan
CC: buildroot@busybox.net
Onderwerp: Re: [Buildroot] [PATCH 1/1] libtirpc : always include svc_auth_none

Sagaert, All,

On 2014-06-14 22:57 +0200, Sagaert Johan spake thusly:
> This function is needed in rpcbind >= 0.2.1 without this patch the 
> linking of rpcbind > 0.2.1 will fail.
> (based on http://www.spinics.net/lists/linux-nfs/msg27277.html )

> Does that fix an autobuilder issue? If so, please add:
Not that I know of, its just needed before I can bump rpcbind to version 0.2.1
I have a few other patches ready to bump rpcbind to 0.2.1, reworked the patches
and added a post install hook to add an rpcbind startup script to /etc/init.d
 

>    Fixes:
>        http://autobuild.b.n/results/XXX/SHA1

What do I need to look for here : http://autobuild.buildroot.net ?

Regards Johan

> Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
> ---
>  ...btirpc-0011-always-include-svc_auth_none.patch | 39 
> ++++++++++++++++++++++
>  1 file changed, 39 insertions(+)
>  create mode 100644 
> package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> 
> diff --git 
> a/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch 
> b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> new file mode 100644
> index 0000000..66c660c
> --- /dev/null
> +++ b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patc
> +++ h
> @@ -0,0 +1,39 @@
> +From 02c9f3898e4189bb0ec817d492b36da6de31abf8 Mon Sep 17 00:00:00 
> +2001
> +From: Sagaert Johan <sagaert.johan@skynet.be>
> +Date: Sat, 14 Jun 2014 22:02:59 +0200
> +Subject: [PATCH 1/1] always include svc_auth_none
> +
> +(needed by rpcbind >=0.2.1 )
> +
> +Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>

Since you use the patch as-is, you should keep the authorship of the patch, and just add your own SoB line.

Regards,
Yann E. MORIN.

> +---
> + src/Makefile.am | 7 +++----
> + 1 file changed, 3 insertions(+), 4 deletions(-)
> +
> +diff --git a/src/Makefile.am b/src/Makefile.am index bd276cf..a6313b7 
> +100644
> +--- a/src/Makefile.am
> ++++ b/src/Makefile.am
> +@@ -48,16 +48,15 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln
> +         getrpcport.c mt_misc.c pmap_clnt.c pmap_getmaps.c pmap_getport.c \
> +         pmap_prot.c pmap_prot2.c pmap_rmt.c rpc_prot.c rpc_commondata.c \
> +         rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \
> +-        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_generic.c \
> +-        svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
> ++        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \
> ++        svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c 
> ++ getpeereid.c
> + 
> + ## XDR
> + libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c 
> + xdr_mem.c xdr_reference.c xdr_stdio.c
> + 
> + ## Secure-RPC
> + if GSS
> +-    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c \
> +-		svc_auth_none.c
> ++    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c
> +     libtirpc_la_LDFLAGS += $(GSSGLUE_LIBS)
> +     libtirpc_la_CFLAGS = -DHAVE_RPCSEC_GSS $(GSSGLUE_CFLAGS)  endif
> +--
> +2.0.0
> +
> --
> 2.0.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

--
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' 
| conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
Yann E. MORIN June 14, 2014, 9:37 p.m. UTC | #3
Sagaert, All,

On 2014-06-14 23:27 +0200, Sagaert Johan spake thusly:
> On 2014-06-14 22:57 +0200, Sagaert Johan spake thusly:
> > This function is needed in rpcbind >= 0.2.1 without this patch the 
> > linking of rpcbind > 0.2.1 will fail.
> > (based on http://www.spinics.net/lists/linux-nfs/msg27277.html )
> 
> > Does that fix an autobuilder issue? If so, please add:
> Not that I know of, its just needed before I can bump rpcbind to version 0.2.1
> I have a few other patches ready to bump rpcbind to 0.2.1, reworked the patches
> and added a post install hook to add an rpcbind startup script to /etc/init.d

> >    Fixes:
> >        http://autobuild.b.n/results/XXX/SHA1
> 
> What do I need to look for here : http://autobuild.buildroot.net ?

http://autobuild.buildroot.net contains all the logs of the autobuilders
that Thomas and Peter run continously. We get a bit over 100 builds per
day, and we can quickly find build failures.

For each build, there is the .config, the end of the build-log and a
few other files, so it is easy to reproduce a build error.

Since your patch looked like it was fixing such an issue, I was
wondering if you had spotted a build failure you fixed with this
patch. But it seems like not.

Regards,
Yann E. MORIN.

> Regards Johan
> 
> > Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
> > ---
> >  ...btirpc-0011-always-include-svc_auth_none.patch | 39 
> > ++++++++++++++++++++++
> >  1 file changed, 39 insertions(+)
> >  create mode 100644 
> > package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> > 
> > diff --git 
> > a/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch 
> > b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
> > new file mode 100644
> > index 0000000..66c660c
> > --- /dev/null
> > +++ b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patc
> > +++ h
> > @@ -0,0 +1,39 @@
> > +From 02c9f3898e4189bb0ec817d492b36da6de31abf8 Mon Sep 17 00:00:00 
> > +2001
> > +From: Sagaert Johan <sagaert.johan@skynet.be>
> > +Date: Sat, 14 Jun 2014 22:02:59 +0200
> > +Subject: [PATCH 1/1] always include svc_auth_none
> > +
> > +(needed by rpcbind >=0.2.1 )
> > +
> > +Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
> 
> Since you use the patch as-is, you should keep the authorship of the patch, and just add your own SoB line.
> 
> Regards,
> Yann E. MORIN.
> 
> > +---
> > + src/Makefile.am | 7 +++----
> > + 1 file changed, 3 insertions(+), 4 deletions(-)
> > +
> > +diff --git a/src/Makefile.am b/src/Makefile.am index bd276cf..a6313b7 
> > +100644
> > +--- a/src/Makefile.am
> > ++++ b/src/Makefile.am
> > +@@ -48,16 +48,15 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln
> > +         getrpcport.c mt_misc.c pmap_clnt.c pmap_getmaps.c pmap_getport.c \
> > +         pmap_prot.c pmap_prot2.c pmap_rmt.c rpc_prot.c rpc_commondata.c \
> > +         rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \
> > +-        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_generic.c \
> > +-        svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
> > ++        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \
> > ++        svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c 
> > ++ getpeereid.c
> > + 
> > + ## XDR
> > + libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c 
> > + xdr_mem.c xdr_reference.c xdr_stdio.c
> > + 
> > + ## Secure-RPC
> > + if GSS
> > +-    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c \
> > +-		svc_auth_none.c
> > ++    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c
> > +     libtirpc_la_LDFLAGS += $(GSSGLUE_LIBS)
> > +     libtirpc_la_CFLAGS = -DHAVE_RPCSEC_GSS $(GSSGLUE_CFLAGS)  endif
> > +--
> > +2.0.0
> > +
> > --
> > 2.0.0
> > 
> > _______________________________________________
> > buildroot mailing list
> > buildroot@busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot
> 
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' 
> | conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'
>
diff mbox

Patch

diff --git a/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
new file mode 100644
index 0000000..66c660c
--- /dev/null
+++ b/package/libtirpc/libtirpc-0011-always-include-svc_auth_none.patch
@@ -0,0 +1,39 @@ 
+From 02c9f3898e4189bb0ec817d492b36da6de31abf8 Mon Sep 17 00:00:00 2001
+From: Sagaert Johan <sagaert.johan@skynet.be>
+Date: Sat, 14 Jun 2014 22:02:59 +0200
+Subject: [PATCH 1/1] always include svc_auth_none
+
+(needed by rpcbind >=0.2.1 )
+
+Signed-off-by: Sagaert Johan <sagaert.johan@skynet.be>
+---
+ src/Makefile.am | 7 +++----
+ 1 file changed, 3 insertions(+), 4 deletions(-)
+
+diff --git a/src/Makefile.am b/src/Makefile.am
+index bd276cf..a6313b7 100644
+--- a/src/Makefile.am
++++ b/src/Makefile.am
+@@ -48,16 +48,15 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln
+         getrpcport.c mt_misc.c pmap_clnt.c pmap_getmaps.c pmap_getport.c \
+         pmap_prot.c pmap_prot2.c pmap_rmt.c rpc_prot.c rpc_commondata.c \
+         rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \
+-        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_generic.c \
+-        svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
++        rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \
++        svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c
+ 
+ ## XDR
+ libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c
+ 
+ ## Secure-RPC
+ if GSS
+-    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c \
+-		svc_auth_none.c
++    libtirpc_la_SOURCES += auth_gss.c authgss_prot.c svc_auth_gss.c
+     libtirpc_la_LDFLAGS += $(GSSGLUE_LIBS)
+     libtirpc_la_CFLAGS = -DHAVE_RPCSEC_GSS $(GSSGLUE_CFLAGS)
+ endif
+-- 
+2.0.0
+