All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
@ 2021-07-22 15:32 Giulio Benetti
  2021-07-22 15:33 ` Giulio Benetti
  0 siblings, 1 reply; 6+ messages in thread
From: Giulio Benetti @ 2021-07-22 15:32 UTC (permalink / raw)
  To: buildroot; +Cc: Giulio Benetti

Add patch to fix time_t build failure on 64-bits platforms.

Fixes:
http://autobuild.buildroot.net/results/9bc1d43a588338b7395af7bc97535ee16a6ea2d9/

Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
---
 ...me_t-build-error-on-64-bits-platform.patch | 52 +++++++++++++++++++
 1 file changed, 52 insertions(+)
 create mode 100644 package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch

diff --git a/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
new file mode 100644
index 0000000000..1dec494f26
--- /dev/null
+++ b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
@@ -0,0 +1,52 @@
+From 9d25faae1216258b617dd2db134bc1d9fa0fd73d Mon Sep 17 00:00:00 2001
+From: Giulio Benetti <giulio.benetti@benettiengineering.com>
+Date: Thu, 22 Jul 2021 16:13:14 +0200
+Subject: [PATCH] nfs-utils: fix time_t build error on 64-bits platforms
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+When passing time_t type to "%ld" on 64-bits platforms where time_t is a
+'long long' we encouter this build failure:
+error: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘time_t’ {aka ‘long long int’} [-Werror=format=]
+
+So let's change "%ld" markers to "%lld" assuming it to be a 64-bits and
+cast variables to '(long long)' if the type is a time_t.
+
+Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
+---
+ utils/nfsdcltrack/nfsdcltrack.c | 2 +-
+ utils/nfsdcltrack/sqlite.c      | 4 ++--
+ 2 files changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/utils/nfsdcltrack/nfsdcltrack.c b/utils/nfsdcltrack/nfsdcltrack.c
+index e926f1c0..437477bb 100644
+--- a/utils/nfsdcltrack/nfsdcltrack.c
++++ b/utils/nfsdcltrack/nfsdcltrack.c
+@@ -525,7 +525,7 @@ cltrack_gracedone(const char *timestr)
+ 	if (*tail)
+ 		return -EINVAL;
+ 
+-	xlog(D_GENERAL, "%s: grace done. gracetime=%ld", __func__, gracetime);
++	xlog(D_GENERAL, "%s: grace done. gracetime=%lld", __func__, (long long)gracetime);
+ 
+ 	ret = sqlite_remove_unreclaimed(gracetime);
+ 
+diff --git a/utils/nfsdcltrack/sqlite.c b/utils/nfsdcltrack/sqlite.c
+index f79aebb3..6e603087 100644
+--- a/utils/nfsdcltrack/sqlite.c
++++ b/utils/nfsdcltrack/sqlite.c
+@@ -544,8 +544,8 @@ sqlite_remove_unreclaimed(time_t grace_start)
+ 	int ret;
+ 	char *err = NULL;
+ 
+-	ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %ld",
+-			grace_start);
++	ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %lld",
++			(long long)grace_start);
+ 	if (ret < 0) {
+ 		return ret;
+ 	} else if ((size_t)ret >= sizeof(buf)) {
+-- 
+2.25.1
+
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
  2021-07-22 15:32 [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build Giulio Benetti
@ 2021-07-22 15:33 ` Giulio Benetti
  2021-07-25 17:40   ` Giulio Benetti
  0 siblings, 1 reply; 6+ messages in thread
From: Giulio Benetti @ 2021-07-22 15:33 UTC (permalink / raw)
  To: buildroot

On 7/22/21 5:32 PM, Giulio Benetti wrote:
> Add patch to fix time_t build failure on 64-bits platforms.

Forgot to add:
```
Patch is pending upstream:
https://patchwork.kernel.org/project/linux-nfs/patch/20210722152411.1156295-1-giulio.benetti@benettiengineering.com/
```
> Fixes:
> http://autobuild.buildroot.net/results/9bc1d43a588338b7395af7bc97535ee16a6ea2d9/
> 
> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> ---
>   ...me_t-build-error-on-64-bits-platform.patch | 52 +++++++++++++++++++
>   1 file changed, 52 insertions(+)
>   create mode 100644 package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
> 
> diff --git a/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
> new file mode 100644
> index 0000000000..1dec494f26
> --- /dev/null
> +++ b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
> @@ -0,0 +1,52 @@
> +From 9d25faae1216258b617dd2db134bc1d9fa0fd73d Mon Sep 17 00:00:00 2001
> +From: Giulio Benetti <giulio.benetti@benettiengineering.com>
> +Date: Thu, 22 Jul 2021 16:13:14 +0200
> +Subject: [PATCH] nfs-utils: fix time_t build error on 64-bits platforms
> +MIME-Version: 1.0
> +Content-Type: text/plain; charset=UTF-8
> +Content-Transfer-Encoding: 8bit
> +
> +When passing time_t type to "%ld" on 64-bits platforms where time_t is a
> +'long long' we encouter this build failure:
> +error: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘time_t’ {aka ‘long long int’} [-Werror=format=]
> +
> +So let's change "%ld" markers to "%lld" assuming it to be a 64-bits and
> +cast variables to '(long long)' if the type is a time_t.
> +
> +Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
> +---
> + utils/nfsdcltrack/nfsdcltrack.c | 2 +-
> + utils/nfsdcltrack/sqlite.c      | 4 ++--
> + 2 files changed, 3 insertions(+), 3 deletions(-)
> +
> +diff --git a/utils/nfsdcltrack/nfsdcltrack.c b/utils/nfsdcltrack/nfsdcltrack.c
> +index e926f1c0..437477bb 100644
> +--- a/utils/nfsdcltrack/nfsdcltrack.c
> ++++ b/utils/nfsdcltrack/nfsdcltrack.c
> +@@ -525,7 +525,7 @@ cltrack_gracedone(const char *timestr)
> + 	if (*tail)
> + 		return -EINVAL;
> +
> +-	xlog(D_GENERAL, "%s: grace done. gracetime=%ld", __func__, gracetime);
> ++	xlog(D_GENERAL, "%s: grace done. gracetime=%lld", __func__, (long long)gracetime);
> +
> + 	ret = sqlite_remove_unreclaimed(gracetime);
> +
> +diff --git a/utils/nfsdcltrack/sqlite.c b/utils/nfsdcltrack/sqlite.c
> +index f79aebb3..6e603087 100644
> +--- a/utils/nfsdcltrack/sqlite.c
> ++++ b/utils/nfsdcltrack/sqlite.c
> +@@ -544,8 +544,8 @@ sqlite_remove_unreclaimed(time_t grace_start)
> + 	int ret;
> + 	char *err = NULL;
> +
> +-	ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %ld",
> +-			grace_start);
> ++	ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %lld",
> ++			(long long)grace_start);
> + 	if (ret < 0) {
> + 		return ret;
> + 	} else if ((size_t)ret >= sizeof(buf)) {
> +--
> +2.25.1
> +
> 

-- 
Giulio Benetti
Benetti Engineering sas
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
  2021-07-22 15:33 ` Giulio Benetti
@ 2021-07-25 17:40   ` Giulio Benetti
  2021-07-25 19:38     ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Giulio Benetti @ 2021-07-25 17:40 UTC (permalink / raw)
  To: buildroot

Please drop this patch since I’m going another upstream patch.

Thank you
Giulio Benetti

> Il giorno 22 lug 2021, alle ore 17:41, Giulio Benetti <giulio.benetti@benettiengineering.com> ha scritto:
> 
> On 7/22/21 5:32 PM, Giulio Benetti wrote:
>> Add patch to fix time_t build failure on 64-bits platforms.
> 
> Forgot to add:
> ```
> Patch is pending upstream:
> https://patchwork.kernel.org/project/linux-nfs/patch/20210722152411.1156295-1-giulio.benetti@benettiengineering.com/
> ```
>> Fixes:
>> http://autobuild.buildroot.net/results/9bc1d43a588338b7395af7bc97535ee16a6ea2d9/
>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> ---
>>  ...me_t-build-error-on-64-bits-platform.patch | 52 +++++++++++++++++++
>>  1 file changed, 52 insertions(+)
>>  create mode 100644 package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
>> diff --git a/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
>> new file mode 100644
>> index 0000000000..1dec494f26
>> --- /dev/null
>> +++ b/package/nfs-utils/0001-nfs-utils-fix-time_t-build-error-on-64-bits-platform.patch
>> @@ -0,0 +1,52 @@
>> +From 9d25faae1216258b617dd2db134bc1d9fa0fd73d Mon Sep 17 00:00:00 2001
>> +From: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> +Date: Thu, 22 Jul 2021 16:13:14 +0200
>> +Subject: [PATCH] nfs-utils: fix time_t build error on 64-bits platforms
>> +MIME-Version: 1.0
>> +Content-Type: text/plain; charset=UTF-8
>> +Content-Transfer-Encoding: 8bit
>> +
>> +When passing time_t type to "%ld" on 64-bits platforms where time_t is a
>> +'long long' we encouter this build failure:
>> +error: format ‘%ld’ expects argument of type ‘long int’, but argument 4 has type ‘time_t’ {aka ‘long long int’} [-Werror=format=]
>> +
>> +So let's change "%ld" markers to "%lld" assuming it to be a 64-bits and
>> +cast variables to '(long long)' if the type is a time_t.
>> +
>> +Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>> +---
>> + utils/nfsdcltrack/nfsdcltrack.c | 2 +-
>> + utils/nfsdcltrack/sqlite.c      | 4 ++--
>> + 2 files changed, 3 insertions(+), 3 deletions(-)
>> +
>> +diff --git a/utils/nfsdcltrack/nfsdcltrack.c b/utils/nfsdcltrack/nfsdcltrack.c
>> +index e926f1c0..437477bb 100644
>> +--- a/utils/nfsdcltrack/nfsdcltrack.c
>> ++++ b/utils/nfsdcltrack/nfsdcltrack.c
>> +@@ -525,7 +525,7 @@ cltrack_gracedone(const char *timestr)
>> +    if (*tail)
>> +        return -EINVAL;
>> +
>> +-    xlog(D_GENERAL, "%s: grace done. gracetime=%ld", __func__, gracetime);
>> ++    xlog(D_GENERAL, "%s: grace done. gracetime=%lld", __func__, (long long)gracetime);
>> +
>> +    ret = sqlite_remove_unreclaimed(gracetime);
>> +
>> +diff --git a/utils/nfsdcltrack/sqlite.c b/utils/nfsdcltrack/sqlite.c
>> +index f79aebb3..6e603087 100644
>> +--- a/utils/nfsdcltrack/sqlite.c
>> ++++ b/utils/nfsdcltrack/sqlite.c
>> +@@ -544,8 +544,8 @@ sqlite_remove_unreclaimed(time_t grace_start)
>> +    int ret;
>> +    char *err = NULL;
>> +
>> +-    ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %ld",
>> +-            grace_start);
>> ++    ret = snprintf(buf, sizeof(buf), "DELETE FROM clients WHERE time < %lld",
>> ++            (long long)grace_start);
>> +    if (ret < 0) {
>> +        return ret;
>> +    } else if ((size_t)ret >= sizeof(buf)) {
>> +--
>> +2.25.1
>> +
> 
> -- 
> Giulio Benetti
> Benetti Engineering sas

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
  2021-07-25 17:40   ` Giulio Benetti
@ 2021-07-25 19:38     ` Petr Vorel
  2021-07-25 19:55       ` Giulio Benetti
  0 siblings, 1 reply; 6+ messages in thread
From: Petr Vorel @ 2021-07-25 19:38 UTC (permalink / raw)
  To: Giulio Benetti; +Cc: buildroot

Hi Giulio,

> Please drop this patch since I’m going another upstream patch.
Yep, I first sent my patches upstream. I should have Cc you, I'll try to
remember for next time.

Also feel free to backport them to Buildroot and please Cc me (so we don't sent
the same patch both).

Kind regards,
Petr
_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
  2021-07-25 19:38     ` Petr Vorel
@ 2021-07-25 19:55       ` Giulio Benetti
  2021-07-25 20:02         ` Petr Vorel
  0 siblings, 1 reply; 6+ messages in thread
From: Giulio Benetti @ 2021-07-25 19:55 UTC (permalink / raw)
  To: Petr Vorel; +Cc: buildroot

Hi Petr,

> Il giorno 25 lug 2021, alle ore 21:38, Petr Vorel <petr.vorel@gmail.com> ha scritto:
> 
> Hi Giulio,
> 
>> Please drop this patch since I’m going another upstream patch.
> Yep, I first sent my patches upstream. I should have Cc you, I'll try to
> remember for next time.

Ah they’re yours, I didn’t have to time to check.

> 
> Also feel free to backport them to Buildroot and please Cc me (so we don't sent
> the same patch both).

Sure, good idea, thank you.

I’m going to backport them to buildroot then.

Best regards
Giulio Benetti

> 
> Kind regards,
> Petr
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build
  2021-07-25 19:55       ` Giulio Benetti
@ 2021-07-25 20:02         ` Petr Vorel
  0 siblings, 0 replies; 6+ messages in thread
From: Petr Vorel @ 2021-07-25 20:02 UTC (permalink / raw)
  To: Giulio Benetti; +Cc: buildroot

Hi Giulio,

> Hi Petr,

> > Il giorno 25 lug 2021, alle ore 21:38, Petr Vorel <petr.vorel@gmail.com> ha scritto:

> > Hi Giulio,

> >> Please drop this patch since I’m going another upstream patch.
> > Yep, I first sent my patches upstream. I should have Cc you, I'll try to
> > remember for next time.

> Ah they’re yours, I didn’t have to time to check.


> > Also feel free to backport them to Buildroot and please Cc me (so we don't sent
> > the same patch both).

> Sure, good idea, thank you.

> I’m going to backport them to buildroot then.

+1. Thanks!

Kind regards,
Petr

> Best regards
> Giulio Benetti


> > Kind regards,
> > Petr
> > _______________________________________________
> > buildroot mailing list
> > buildroot@busybox.net
> > http://lists.busybox.net/mailman/listinfo/buildroot

_______________________________________________
buildroot mailing list
buildroot@busybox.net
http://lists.busybox.net/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-07-25 20:03 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-22 15:32 [Buildroot] [PATCH] package/nfs-utils: fix time_t on 64-bits platforms build Giulio Benetti
2021-07-22 15:33 ` Giulio Benetti
2021-07-25 17:40   ` Giulio Benetti
2021-07-25 19:38     ` Petr Vorel
2021-07-25 19:55       ` Giulio Benetti
2021-07-25 20:02         ` Petr Vorel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.