* [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
@ 2023-03-25 17:58 Raphael Pavlidis
2023-03-26 12:18 ` Thomas Petazzoni via buildroot
2023-07-29 22:15 ` Thomas Petazzoni via buildroot
0 siblings, 2 replies; 6+ messages in thread
From: Raphael Pavlidis @ 2023-03-25 17:58 UTC (permalink / raw)
To: buildroot; +Cc: Fabrice Fontaine, Raphael Pavlidis
Add a patch to fix the build if only yescrypt is enabled.
Fixes:
- http://autobuild.buildroot.net/results/df6a16a55bf02cf2d7377f9636913f0b3bca574f
- http://autobuild.buildroot.net/results/758d4fd297fbd77ef846d3a3d4c207021f866752
Signed-off-by: Raphael Pavlidis <raphael.pavlidis@gmail.com>
---
...-sflg-for-USE_YESCRYPT-in-newusers.c.patch | 29 +++++++++++++++++++
1 file changed, 29 insertions(+)
create mode 100644 package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
diff --git a/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch b/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
new file mode 100644
index 0000000000..6c2a15d898
--- /dev/null
+++ b/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
@@ -0,0 +1,29 @@
+From 8f7d5e8adae3aa16c0e3a4cb5db34d6764aa0495 Mon Sep 17 00:00:00 2001
+From: Raphael Pavlidis <raphael.pavlidis@gmail.com>
+Date: Thu, 23 Mar 2023 19:16:12 +0100
+Subject: [PATCH v1 1/1] Define sflg for USE_YESCRYPT in newusers.c
+
+If only yescrypt is used then the boolean sflg is not defined, but it is
+access. Therefore, it causes a build error.
+
+Signed-off-by: Raphael Pavlidis <raphael.pavlidis@gmail.com>
+---
+ src/newusers.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/newusers.c b/src/newusers.c
+index dd6b4421..9c2a626a 100644
+--- a/src/newusers.c
++++ b/src/newusers.c
+@@ -60,7 +60,7 @@ static bool rflg = false; /* create a system account */
+ #ifndef USE_PAM
+ static /*@null@*//*@observer@*/char *crypt_method = NULL;
+ #define cflg (NULL != crypt_method)
+-#if defined(USE_SHA_CRYPT) || defined(USE_BCRYPT)
++#if defined(USE_SHA_CRYPT) || defined(USE_BCRYPT) || defined(USE_YESCRYPT)
+ static bool sflg = false;
+ #endif
+ #ifdef USE_SHA_CRYPT
+--
+2.39.2
+
--
2.39.2
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
2023-03-25 17:58 [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build Raphael Pavlidis
@ 2023-03-26 12:18 ` Thomas Petazzoni via buildroot
2023-04-04 18:31 ` Raphael Pavlidis
2023-07-29 22:15 ` Thomas Petazzoni via buildroot
1 sibling, 1 reply; 6+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-03-26 12:18 UTC (permalink / raw)
To: Raphael Pavlidis; +Cc: Fabrice Fontaine, buildroot
On Sat, 25 Mar 2023 18:58:15 +0100
Raphael Pavlidis <raphael.pavlidis@gmail.com> wrote:
> Add a patch to fix the build if only yescrypt is enabled.
>
> Fixes:
> - http://autobuild.buildroot.net/results/df6a16a55bf02cf2d7377f9636913f0b3bca574f
> - http://autobuild.buildroot.net/results/758d4fd297fbd77ef846d3a3d4c207021f866752
> Signed-off-by: Raphael Pavlidis <raphael.pavlidis@gmail.com>
> ---
> ...-sflg-for-USE_YESCRYPT-in-newusers.c.patch | 29 +++++++++++++++++++
> 1 file changed, 29 insertions(+)
> create mode 100644 package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
>
> diff --git a/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch b/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
> new file mode 100644
> index 0000000000..6c2a15d898
> --- /dev/null
> +++ b/package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
> @@ -0,0 +1,29 @@
> +From 8f7d5e8adae3aa16c0e3a4cb5db34d6764aa0495 Mon Sep 17 00:00:00 2001
> +From: Raphael Pavlidis <raphael.pavlidis@gmail.com>
> +Date: Thu, 23 Mar 2023 19:16:12 +0100
> +Subject: [PATCH v1 1/1] Define sflg for USE_YESCRYPT in newusers.c
> +
> +If only yescrypt is used then the boolean sflg is not defined, but it is
> +access. Therefore, it causes a build error.
> +
> +Signed-off-by: Raphael Pavlidis <raphael.pavlidis@gmail.com>
Thanks for fixing this! Has this patch been submitted to the upstream
shadow project?
Thanks!
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
2023-03-26 12:18 ` Thomas Petazzoni via buildroot
@ 2023-04-04 18:31 ` Raphael Pavlidis
2023-05-26 19:16 ` Raphael Pavlidis
[not found] ` <eb8cf642-62ed-3f32-a32e-f7d28fd6998f__26834.4457073667$1685128600$gmane$org@gmail.com>
0 siblings, 2 replies; 6+ messages in thread
From: Raphael Pavlidis @ 2023-04-04 18:31 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: Fabrice Fontaine, buildroot
Hi Thomas,
On 26.03.23 14:18, Thomas Petazzoni wrote:
> On Sat, 25 Mar 2023 18:58:15 +0100
> Raphael Pavlidis <raphael.pavlidis@gmail.com> wrote:
>
[SNIP]
>
> Thanks for fixing this! Has this patch been submitted to the upstream
> shadow project?
>
Not yet.
> Thanks!
>
> Thomas
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
2023-04-04 18:31 ` Raphael Pavlidis
@ 2023-05-26 19:16 ` Raphael Pavlidis
[not found] ` <eb8cf642-62ed-3f32-a32e-f7d28fd6998f__26834.4457073667$1685128600$gmane$org@gmail.com>
1 sibling, 0 replies; 6+ messages in thread
From: Raphael Pavlidis @ 2023-05-26 19:16 UTC (permalink / raw)
To: Thomas Petazzoni; +Cc: Fabrice Fontaine, buildroot
Hi Thomas,
[SNIP]
>
>>
>> Thanks for fixing this! Has this patch been submitted to the upstream
>> shadow project?
>>
It seems that shadow is hosted on GitHub. Currently, I do not have a
GitHub account and do not want one. Feel free (and everybody else) to
take the patch and make a PR.
[SNIP]
Thanks,
Raphael Pavlidis
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
[not found] ` <eb8cf642-62ed-3f32-a32e-f7d28fd6998f__26834.4457073667$1685128600$gmane$org@gmail.com>
@ 2023-07-14 16:54 ` Bernd Kuhls
0 siblings, 0 replies; 6+ messages in thread
From: Bernd Kuhls @ 2023-07-14 16:54 UTC (permalink / raw)
To: buildroot
Am Fri, 26 May 2023 21:16:11 +0200 schrieb Raphael Pavlidis:
> It seems that shadow is hosted on GitHub. Currently, I do not have a
> GitHub account and do not want one. Feel free (and everybody else) to
> take the patch and make a PR.
Hi Raphael,
sorry, i did not see your patch when I created mine.
Upstream in the meantime committed my patch
https://github.com/shadow-maint/shadow/commit/
29da702491eea314b915ea9c7a83c9af80cf5797
Regards, Bernd
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build
2023-03-25 17:58 [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build Raphael Pavlidis
2023-03-26 12:18 ` Thomas Petazzoni via buildroot
@ 2023-07-29 22:15 ` Thomas Petazzoni via buildroot
1 sibling, 0 replies; 6+ messages in thread
From: Thomas Petazzoni via buildroot @ 2023-07-29 22:15 UTC (permalink / raw)
To: Raphael Pavlidis; +Cc: Fabrice Fontaine, buildroot
On Sat, 25 Mar 2023 18:58:15 +0100
Raphael Pavlidis <raphael.pavlidis@gmail.com> wrote:
> Add a patch to fix the build if only yescrypt is enabled.
>
> Fixes:
> - http://autobuild.buildroot.net/results/df6a16a55bf02cf2d7377f9636913f0b3bca574f
> - http://autobuild.buildroot.net/results/758d4fd297fbd77ef846d3a3d4c207021f866752
> Signed-off-by: Raphael Pavlidis <raphael.pavlidis@gmail.com>
> ---
> ...-sflg-for-USE_YESCRYPT-in-newusers.c.patch | 29 +++++++++++++++++++
> 1 file changed, 29 insertions(+)
> create mode 100644 package/shadow/0001-Define-sflg-for-USE_YESCRYPT-in-newusers.c.patch
I have applied Bernd's patch instead, which has been accepted upstream.
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-07-29 22:15 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-25 17:58 [Buildroot] [PATCH v1 1/1] package/shadow: only yescrypt build Raphael Pavlidis
2023-03-26 12:18 ` Thomas Petazzoni via buildroot
2023-04-04 18:31 ` Raphael Pavlidis
2023-05-26 19:16 ` Raphael Pavlidis
[not found] ` <eb8cf642-62ed-3f32-a32e-f7d28fd6998f__26834.4457073667$1685128600$gmane$org@gmail.com>
2023-07-14 16:54 ` Bernd Kuhls
2023-07-29 22:15 ` Thomas Petazzoni via buildroot
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox