Skip to contentSkip to conversation

libtirpc: Bump to version 1.1.4

PATCH New Petr Vorel 2 participants 5 messages Nov 17, 2018 – Nov 19, 2018

lore
Share & tools
lore

Petr Vorel

author
#0 lore raw

PATCH 1/2 libtirpc: Bump to version 1.1.4

Signed-off-by Petr Vorel <[email protected]>

---
Changes v1->v2:
Add sha1 from sourceforge.

PS: Baruch, thx for info about hash location on sourceforge.

---
 package/libtirpc/libtirpc.hash | 4 ++--
 package/libtirpc/libtirpc.mk   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
package/libtirpc/libtirpc.hash +2 -2
index 643a5d2050..eacc94d1de 100644
@@ -1,5 +1,5 @@
 # From sourceforge's info on download page:
-sha1 48adb32dc7c3b73c66f001c239da76b8398abf11  libtirpc-1.0.3.tar.bz2
+sha1 d85717035cb9bd6c45557a1eb1351d3af9a69ff7  libtirpc-1.1.4.tar.bz2
 # Locally computed
-sha256 86c3a78fc1bddefa96111dd233124c703b22a78884203c55c3e06b3be6a0fd5e  libtirpc-1.0.3.tar.bz2
+sha256 2ca529f02292e10c158562295a1ffd95d2ce8af97820e3534fe1b0e3aec7561d  libtirpc-1.1.4.tar.bz2
 sha256 17cf6098f95bdbb269f0bbc68e76c88fe20487ca7ec53f454923ab4256ecd2e7  COPYING
package/libtirpc/libtirpc.mk +1 -1
index 36f9f7ef21..1e1905b713 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LIBTIRPC_VERSION = 1.0.3
+LIBTIRPC_VERSION = 1.1.4
 LIBTIRPC_SOURCE = libtirpc-$(LIBTIRPC_VERSION).tar.bz2
 LIBTIRPC_SITE = http://downloads.sourceforge.net/project/libtirpc/libtirpc/$(LIBTIRPC_VERSION)
 LIBTIRPC_LICENSE = BSD-3-Clause
Signature · 1 line
2.19.1
author
#2 lore raw

PATCH 2/2 libtirpc: Fix whitespace on 4th patch

In reply to Petr Vorel
Signed-off-by Petr Vorel <[email protected]>

---
Should be the same as v1.

---
 ...es-needed-to-build-rpcbind-on-top-of.patch | 78 +++++++++----------
 1 file changed, 38 insertions(+), 40 deletions(-)
package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch +38 -40
index 7f53ffca16..797fb7a42c 100644
@@ -22,16 +22,16 @@ index aa5908e..4999066 100644
 +++ b/Makefile.am
 @@ -2,6 +2,9 @@ SUBDIRS = src man doc rpcgen
  ACLOCAL_AMFLAGS = -I m4
- 
+
  GENFILES	       = tirpc/rpcsvc/crypt.h \
 +			 tirpc/rpcsvc/mount.h \
 +			 tirpc/rpcsvc/nfs_prot.h \
 +			 tirpc/rpcsvc/rquota.h \
  			 tirpc/rpc/rpcb_prot.h
- 
+
  noinst_HEADERS	       = tirpc/reentrant.h \
 @@ -11,6 +14,9 @@ noinst_HEADERS	       = tirpc/reentrant.h \
- 
+
  nobase_include_HEADERS = tirpc/netconfig.h \
  			 tirpc/rpcsvc/crypt.x \
 +			 tirpc/rpcsvc/mount.x \

@@ -53,23 +53,23 @@ index 0000000..f68a06f
 + * may copy or modify Sun RPC without charge, but are not authorized
 + * to license or distribute it to anyone else except as part of a product or
 + * program developed by the user.
-+ * 
++ *
 + * SUN RPC IS PROVIDED AS IS WITH NO WARRANTIES OF ANY KIND INCLUDING THE
 + * WARRANTIES OF DESIGN, MERCHANTIBILITY AND FITNESS FOR A PARTICULAR
 + * PURPOSE, OR ARISING FROM A COURSE OF DEALING, USAGE OR TRADE PRACTICE.
-+ * 
++ *
 + * Sun RPC is provided with no support and without any obligation on the
 + * part of Sun Microsystems, Inc. to assist in its use, correction,
 + * modification or enhancement.
-+ * 
++ *
 + * SUN MICROSYSTEMS, INC. SHALL HAVE NO LIABILITY WITH RESPECT TO THE
 + * INFRINGEMENT OF COPYRIGHTS, TRADE SECRETS OR ANY PATENTS BY SUN RPC
 + * OR ANY PART THEREOF.
-+ * 
++ *
 + * In no event will Sun Microsystems, Inc. be liable for any lost revenue
 + * or profits or other special, indirect and consequential damages, even if
 + * Sun has been advised of the possibility of such damages.
-+ * 
++ *
 + * Sun Microsystems, Inc.
 + * 2550 Garcia Avenue
 + * Mountain View, California  94043

@@ -101,13 +101,13 @@ index 0000000..f68a06f
 + * or a directory. The file handle can contain whatever information the
 + * server needs to distinguish an individual file.
 + */
-+typedef opaque fhandle[FHSIZE];	
++typedef opaque fhandle[FHSIZE];
 +#ifdef WANT_NFS3
 +typedef opaque fhandle3<FHSIZE3>;
 +#endif
 +
 +/*
-+ * If a status of zero is returned, the call completed successfully, and 
++ * If a status of zero is returned, the call completed successfully, and
 + * a file handle for the directory follows. A non-zero status indicates
 + * some sort of error. The status corresponds with UNIX error numbers.
 + */

@@ -203,20 +203,20 @@ index 0000000..f68a06f
 +		void
 +		MOUNTPROC_NULL(void) = 0;
 +
-+		/*	
++		/*
 +		 * If fhs_status is 0, then fhs_fhandle contains the
-+	 	 * file handle for the directory. This file handle may
++		 * file handle for the directory. This file handle may
 +		 * be used in the NFS protocol. This procedure also adds
 +		 * a new entry to the mount list for this client mounting
 +		 * the directory.
 +		 * Unix authentication required.
 +		 */
-+		fhstatus 
++		fhstatus
 +		MOUNTPROC_MNT(dirpath) = 1;
 +
 +		/*
-+		 * Returns the list of remotely mounted filesystems. The 
-+		 * mountlist contains one entry for each hostname and 
++		 * Returns the list of remotely mounted filesystems. The
++		 * mountlist contains one entry for each hostname and
 +		 * directory pair.
 +		 */
 +		mountlist

@@ -273,8 +273,8 @@ index 0000000..f68a06f
 +		MOUNTPROC_MNT(dirpath) = 1;
 +
 +		/*
-+		 * Returns the list of remotely mounted filesystems. The 
-+		 * mountlist contains one entry for each hostname and 
++		 * Returns the list of remotely mounted filesystems. The
++		 * mountlist contains one entry for each hostname and
 +		 * directory pair.
 +		 */
 +		mountlist

@@ -308,7 +308,7 @@ new file mode 100644
 index 0000000..01564f8
 --- /dev/null
 +++ b/tirpc/rpcsvc/nfs_prot.x
-@@ -0,0 +1,1266 @@
+@@ -0,0 +1,1265 @@
 +/*
 + * Sun RPC is a product of Sun Microsystems, Inc. and is provided for
 + * unrestricted use provided that this legend is included on all tape

@@ -316,23 +316,23 @@ index 0000000..01564f8
 + * may copy or modify Sun RPC without charge, but are not authorized
 + * to license or distribute it to anyone else except as part of a product or
 + * program developed by the user.
-+ * 
++ *
 + * SUN RPC IS PROVIDED AS IS WITH NO WARRANTIES OF ANY KIND INCLUDING THE
 + * WARRANTIES OF DESIGN, MERCHANTIBILITY AND FITNESS FOR A PARTICULAR
 + * PURPOSE, OR ARISING FROM A COURSE OF DEALING, USAGE OR TRADE PRACTICE.
-+ * 
++ *
 + * Sun RPC is provided with no support and without any obligation on the
 + * part of Sun Microsystems, Inc. to assist in its use, correction,
 + * modification or enhancement.
-+ * 
++ *
 + * SUN MICROSYSTEMS, INC. SHALL HAVE NO LIABILITY WITH RESPECT TO THE
 + * INFRINGEMENT OF COPYRIGHTS, TRADE SECRETS OR ANY PATENTS BY SUN RPC
 + * OR ANY PART THEREOF.
-+ * 
++ *
 + * In no event will Sun Microsystems, Inc. be liable for any lost revenue
 + * or profits or other special, indirect and consequential damages, even if
 + * Sun has been advised of the possibility of such damages.
-+ * 
++ *
 + * Sun Microsystems, Inc.
 + * 2550 Garcia Avenue
 + * Mountain View, California  94043

@@ -413,7 +413,7 @@ index 0000000..01564f8
 +	opaque data[NFS_FHSIZE];
 +};
 +
-+/* 
++/*
 + * Timeval
 + */
 +struct nfstime {

@@ -455,7 +455,7 @@ index 0000000..01564f8
 +};
 +
 +
-+typedef string filename<NFS_MAXNAMLEN>; 
++typedef string filename<NFS_MAXNAMLEN>;
 +typedef string nfspath<NFS_MAXPATHLEN>;
 +
 +/*

@@ -529,7 +529,7 @@ index 0000000..01564f8
 +};
 +
 +/*
-+ * Arguments to remote write 
++ * Arguments to remote write
 + */
 +struct writeargs {
 +	nfs_fh	file;		/* handle for file */

@@ -694,7 +694,7 @@ index 0000000..01564f8
 +	opaque data<NFS3_FHSIZE>;
 +};
 +
-+/* 
++/*
 + * Timeval (v3)
 + */
 +struct nfstime3 {

@@ -1448,28 +1448,28 @@ index 0000000..01564f8
 + */
 +program NFS_PROGRAM {
 +	version NFS_VERSION {
-+		void 
++		void
 +		NFSPROC_NULL(void) = 0;
 +
-+		attrstat 
++		attrstat
 +		NFSPROC_GETATTR(nfs_fh) =	1;
 +
-+		attrstat 
++		attrstat
 +		NFSPROC_SETATTR(sattrargs) = 2;
 +
-+		void 
++		void
 +		NFSPROC_ROOT(void) = 3;
 +
-+		diropres 
++		diropres
 +		NFSPROC_LOOKUP(diropargs) = 4;
 +
-+		readlinkres 
++		readlinkres
 +		NFSPROC_READLINK(nfs_fh) = 5;
 +
-+		readres 
++		readres
 +		NFSPROC_READ(readargs) = 6;
 +
-+		void 
++		void
 +		NFSPROC_WRITECACHE(void) = 7;
 +
 +		attrstat

@@ -1574,7 +1574,6 @@ index 0000000..01564f8
 +	} = 3;
 +} = 100003;
 +#endif
-+
 diff --git a/tirpc/rpcsvc/rquota.x b/tirpc/rpcsvc/rquota.x
 new file mode 100644
 index 0000000..72864d1

@@ -1616,7 +1615,7 @@ index 0000000..72864d1
 +	unsigned int rq_curfiles;	/* current # allocated files */
 +	unsigned int rq_btimeleft;	/* time left for excessive disk use */
 +	unsigned int rq_ftimeleft;	/* time left for excessive files */
-+};	
++};
 +
 +enum gqr_status {
 +	Q_OK = 1,		/* quota returned */

@@ -1642,12 +1641,11 @@ index 0000000..72864d1
 +		RQUOTAPROC_GETQUOTA(getquota_args) = 1;
 +
 +		/*
-+	 	 * Get active quotas only
++		 * Get active quotas only
 +		 */
 +		getquota_rslt
 +		RQUOTAPROC_GETACTIVEQUOTA(getquota_args) = 2;
 +	} = 1;
 +} = 100011;
--- 
+--
 1.9.1
-
Signature · 1 line
2.19.1
#3 lore raw

PATCH 2/2 libtirpc: Fix whitespace on 4th patch

Hello,

Quoted 12 lines · Petr Vorel
Signed-off-by Petr Vorel <[email protected]>

---
Should be the same as v1.

---
 ...es-needed-to-build-rpcbind-on-top-of.patch | 78 +++++++++----------
 1 file changed, 38 insertions(+), 40 deletions(-)
package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch +0 -0
index 7f53ffca16..797fb7a42c 100644

package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch index aa5908e..4999066 100644 · line 25

Quoted 5 lines
package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch +1 -1
@@ -22,16 +22,16 @@ index aa5908e..4999066 100644
 +++ b/Makefile.am
 @@ -2,6 +2,9 @@ SUBDIRS = src man doc rpcgen
  ACLOCAL_AMFLAGS = -I m4
- 
+

These are additional changes, i.e you are removing whitespace that is here in the original source code.

package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch index aa5908e..4999066 100644 · line 31

Quoted 7 lines
package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch +1 -1
  GENFILES	       = tirpc/rpcsvc/crypt.h \
 +			 tirpc/rpcsvc/mount.h \
 +			 tirpc/rpcsvc/nfs_prot.h \
 +			 tirpc/rpcsvc/rquota.h \
  			 tirpc/rpc/rpcb_prot.h
- 
+

Ditto.

package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch index aa5908e..4999066 100644 · line 34

Quoted 4 lines
package/libtirpc/0004-Add-more-XDR-files-needed-to-build-rpcbind-on-top-of.patch +1 -1
  noinst_HEADERS	       = tirpc/reentrant.h \
 @@ -11,6 +14,9 @@ noinst_HEADERS	       = tirpc/reentrant.h \
- 
+

Ditto.

I.e, in the Makefile.am, our patch is not adding any more trailing whitespace.

Then, in the .x files, those are copy/pasted from I don't remember which project. I'm not sure how much sense it makes to fix trailing whitespace in those patches.

Instead, it would be tremendously more useful to work with libtirpc upstream to get all of those patches merged. I think I tried to do so a number of years ago, but without much luck.

Best regards,

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com