Skip to contentSkip to conversation

package/libnfs: bump to version 3.0.0

PATCH Superseded Jörg Krause 2 participants 4 messages Nov 5, 2018

v2 of 3 Superseded Latest is v3 →

lore
Share & tools
lore

Jörg Krause

author
#0 lore raw

PATCH v2 1/1 package/libnfs: bump to version 3.0.0

Also add an patch from upstream to fix building libnfs with the musl C library. This issue was introduced upstream between version 2.0.0 and version 3.0.0.

Note, that upstreams commit message says it fixes a warning. However, as musl is more strictly regarding missing headers it actually fixes the build with musl.

Signed-off-by J?rg Krause <[email protected]>

---
v1 -> v2:
- fix hash of license file COPYING

---
 ...-libnfs.c-include-time.h-in-libnfs.c.patch | 35 +++++++++++++++++++
 package/libnfs/libnfs.hash                    |  4 +--
 package/libnfs/libnfs.mk                      |  2 +-
 3 files changed, 38 insertions(+), 3 deletions(-)
 create mode 100644 package/libnfs/0002-libnfs.c-include-time.h-in-libnfs.c.patch
package/libnfs/0002-libnfs.c-include-time.h-in-libnfs.c.patch new file +35 -0
new file mode 100644
index 0000000000..34e584017e
@@ -0,0 +1,35 @@
+From b56bd3d99f67a3273fec3c0c5d9560413465a1db Mon Sep 17 00:00:00 2001
+From: Anna Lyons <[email protected]>
+Date: Thu, 12 Jul 2018 17:45:23 +1000
+Subject: [PATCH] libnfs.c: include <time.h> in libnfs.c
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+time() is used in this file, which is from time.h. Fixes a warning.
+
+Note, that building with musl breaks otherwise, as it is more strict regarding
+missing header inclusions.
+
+Backported from: 794a29f01f8e5961adc5c79d5ba5ac1062877e6c
+
+Signed-off-by: J?rg Krause <[email protected]>
+---
+ lib/libnfs.c | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/lib/libnfs.c b/lib/libnfs.c
+index d89d84d..43c17b9 100755
+--- a/lib/libnfs.c
++++ b/lib/libnfs.c
+@@ -70,6 +70,7 @@
+ #include <stdarg.h>
+ #include <stdlib.h>
+ #include <string.h>
++#include <time.h>
+ #include "slist.h"
+ #include "libnfs.h"
+ #include "libnfs-raw.h"
+-- 
+2.19.1
+
package/libnfs/libnfs.hash +2 -2
index 2bd8caf255..0e4dc6fbc9 100644
@@ -1,6 +1,6 @@
 # Locally calculated
-sha256 7ea6cd8fa6c461d01091e584d424d28e137d23ff4b65b95d01a3fd0ef95d120e  libnfs-libnfs-2.0.0.tar.gz
-sha256 2d152e3a2f31ef0fe14d4908377277f8215fb5c82ec9329d1eed081c845fc85f  COPYING
+sha256 445d92c5fc55e4a5b115e358e60486cf8f87ee50e0103d46a02e7fb4618566a5  libnfs-libnfs-3.0.0.tar.gz
+sha256 edd960c0142b8ada98b43b6396b78f4e557b0bc70ac601a51e397ad04070e2c5  COPYING
 sha256 d9406ced95457941032aa11d04623b8ab71f2827a3395ebef137aec475be35b1  LICENCE-BSD.txt
 sha256 dc626520dcd53a22f727af3ee42c770e56c97a64fe3adb063799d8ab032fe551  LICENCE-LGPL-2.1.txt
 sha256 8ceb4b9ee5adedde47b31e975c1d90c73ad27b6b165a1dcd80c7c545eb65b903  LICENCE-GPL-3.txt
package/libnfs/libnfs.mk +1 -1
index 0cf07d2143..3a84e0aafa 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-LIBNFS_VERSION = libnfs-2.0.0
+LIBNFS_VERSION = libnfs-3.0.0
 LIBNFS_SITE = $(call github,sahlberg,libnfs,$(LIBNFS_VERSION))
 LIBNFS_INSTALL_STAGING = YES
 LIBNFS_AUTORECONF = YES
Signature · 1 line
2.19.1
#1 lore raw

PATCH v2 1/1 package/libnfs: bump to version 3.0.0

Hello,

Quoted 4 lines · J?rg Krause

Also add an patch from upstream to fix building libnfs with the musl C library. This issue was introduced upstream between version 2.0.0 and version 3.0.0.

Quoted 8 lines

Note, that upstreams commit message says it fixes a warning. However, as musl is more strictly regarding missing headers it actually fixes the build with musl.

Signed-off-by J?rg Krause <[email protected]>

---
v1 -> v2:
- fix hash of license file COPYING

What are the changes in the COPYING file ?

Thanks,

Thomas

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

PATCH v2 1/1 package/libnfs: bump to version 3.0.0

Quoted 11 lines · Thomas Petazzoni

Hello,

Quoted 8 lines · J?rg Krause

Also add an patch from upstream to fix building libnfs with the musl C library. This issue was introduced upstream between version 2.0.0 and version 3.0.0.

Note, that upstreams commit message says it fixes a warning. However, as musl is more strictly regarding missing headers it actually fixes the build with musl.

Quoted 6 lines
Quoted 4 lines
Signed-off-by J?rg Krause <[email protected]>

---
v1 -> v2:
- fix hash of license file COPYING

What are the changes in the COPYING file ?

I contains a clarification about the .x files being destributed under the simplified BSD licence.

We already note in LIBNFS_LICENSE that the .x files are BSD-2-Clause.

Shall I send a v3?

J?rg

#3 lore raw

PATCH v2 1/1 package/libnfs: bump to version 3.0.0

Hello,

Quoted 17 lines · J?rg Krause
Quoted 16 lines · Thomas Petazzoni

Hello,

Quoted 12 lines · J?rg Krause

Also add an patch from upstream to fix building libnfs with the musl C library. This issue was introduced upstream between version 2.0.0 and version 3.0.0.

Note, that upstreams commit message says it fixes a warning. However, as musl is more strictly regarding missing headers it actually fixes the build with musl.

Signed-off-by J?rg Krause <[email protected]>

---
v1 -> v2:
- fix hash of license file COPYING

Quoted 8 lines
Quoted 1 line

What are the changes in the COPYING file ?

I contains a clarification about the .x files being destributed under the simplified BSD licence.

We already note in LIBNFS_LICENSE that the .x files are BSD-2-Clause.

Shall I send a v3?

When the hash of a license file changes, it is always good to explain briefly in the commit log what changed, as it makes it clear that you have paid attention at the changes and it was not just "let's update the hash to whatever it needs to be to make legal-info happy" :-)

Thanks!

Thomas

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