All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
@ 2021-09-05 17:34 Yann E. MORIN
  2021-09-05 17:44 ` Yann E. MORIN
  2021-09-06  9:07 ` Michael Walle
  0 siblings, 2 replies; 7+ messages in thread
From: Yann E. MORIN @ 2021-09-05 17:34 UTC (permalink / raw)
  To: buildroot; +Cc: Leon de Rooij, Kevin Tomary, Yann E. MORIN

Version 1.46.3 had a regression, which meant the file that would store
the filesystem image had to pre-exist, or mkfs.ext2 would fail to
generate the filesystem:

    mkfs.ext4: No such file or directory while trying to determine
    filesystem size

The regression was fixed upstream, and is now part of the 1.46.4
release, so bump to that release.

Fixes: #14196

Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
Reported-by: Leon de Rooij <leon@exquisip.nl>

---
For the stable branch 2021.08.x, I'll send a different patch, that just
backports the upstream fix.
---
 package/e2fsprogs/e2fsprogs.hash | 2 +-
 package/e2fsprogs/e2fsprogs.mk   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
index 3661801199..63c7c9929f 100644
--- a/package/e2fsprogs/e2fsprogs.hash
+++ b/package/e2fsprogs/e2fsprogs.hash
@@ -1,5 +1,5 @@
 # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
-sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
+sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
 # Locally calculated
 sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
 sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
index 5528ecbc34..ca891066a2 100644
--- a/package/e2fsprogs/e2fsprogs.mk
+++ b/package/e2fsprogs/e2fsprogs.mk
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-E2FSPROGS_VERSION = 1.46.3
+E2FSPROGS_VERSION = 1.46.4
 E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
 E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
 E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
-- 
2.25.1

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-05 17:34 [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression Yann E. MORIN
@ 2021-09-05 17:44 ` Yann E. MORIN
  2021-09-08 19:54   ` Romain Naour
  2021-09-06  9:07 ` Michael Walle
  1 sibling, 1 reply; 7+ messages in thread
From: Yann E. MORIN @ 2021-09-05 17:44 UTC (permalink / raw)
  To: buildroot; +Cc: Leon de Rooij, Kevin Tomary

All,

On 2021-09-05 19:34 +0200, Yann E. MORIN spake thusly:
> Version 1.46.3 had a regression, which meant the file that would store
> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
> generate the filesystem:
> 
>     mkfs.ext4: No such file or directory while trying to determine
>     filesystem size
> 
> The regression was fixed upstream, and is now part of the 1.46.4
> release, so bump to that release.
> 
> Fixes: #14196
> 
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
> Reported-by: Leon de Rooij <leon@exquisip.nl>
> 
> ---
> For the stable branch 2021.08.x, I'll send a different patch, that just
> backports the upstream fix.

In fact, 2021.08 still has e2fsprogs 1.46.2, which is not affected; so,
no backports needed.

Regards,
Yann E. MORIN.

> ---
>  package/e2fsprogs/e2fsprogs.hash | 2 +-
>  package/e2fsprogs/e2fsprogs.mk   | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
> index 3661801199..63c7c9929f 100644
> --- a/package/e2fsprogs/e2fsprogs.hash
> +++ b/package/e2fsprogs/e2fsprogs.hash
> @@ -1,5 +1,5 @@
>  # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
> -sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
> +sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
>  # Locally calculated
>  sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
>  sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
> index 5528ecbc34..ca891066a2 100644
> --- a/package/e2fsprogs/e2fsprogs.mk
> +++ b/package/e2fsprogs/e2fsprogs.mk
> @@ -4,7 +4,7 @@
>  #
>  ################################################################################
>  
> -E2FSPROGS_VERSION = 1.46.3
> +E2FSPROGS_VERSION = 1.46.4
>  E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
>  E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
>  E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
> -- 
> 2.25.1
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-05 17:34 [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression Yann E. MORIN
  2021-09-05 17:44 ` Yann E. MORIN
@ 2021-09-06  9:07 ` Michael Walle
  1 sibling, 0 replies; 7+ messages in thread
From: Michael Walle @ 2021-09-06  9:07 UTC (permalink / raw)
  To: Yann E. MORIN; +Cc: Kevin Tomary, Leon de Rooij, buildroot

Am 2021-09-05 19:34, schrieb Yann E. MORIN:
> Version 1.46.3 had a regression, which meant the file that would store
> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
> generate the filesystem:
> 
>     mkfs.ext4: No such file or directory while trying to determine
>     filesystem size
> 
> The regression was fixed upstream, and is now part of the 1.46.4
> release, so bump to that release.
> 
> Fixes: #14196
> 
> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
> Reported-by: Leon de Rooij <leon@exquisip.nl>

Tested-by: Michael Walle <michael@walle.cc>
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-05 17:44 ` Yann E. MORIN
@ 2021-09-08 19:54   ` Romain Naour
  2021-09-08 20:22     ` Romain Naour
  2021-09-08 20:22     ` Romain Naour
  0 siblings, 2 replies; 7+ messages in thread
From: Romain Naour @ 2021-09-08 19:54 UTC (permalink / raw)
  To: Yann E. MORIN, buildroot; +Cc: Kevin Tomary, Leon de Rooij

Hello Yann, All,

Le 05/09/2021 à 19:44, Yann E. MORIN a écrit :
> All,
> 
> On 2021-09-05 19:34 +0200, Yann E. MORIN spake thusly:
>> Version 1.46.3 had a regression, which meant the file that would store
>> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
>> generate the filesystem:
>>
>>     mkfs.ext4: No such file or directory while trying to determine
>>     filesystem size
>>
>> The regression was fixed upstream, and is now part of the 1.46.4
>> release, so bump to that release.
>>
>> Fixes: #14196
>>
>> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
>> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
>> Reported-by: Leon de Rooij <leon@exquisip.nl>

There is another regression with e2fsprog 1.46.4:

https://gitlab.com/kubu93/buildroot/-/jobs/1568672940

With the fix backported to 1.46.3 the ext4 test is ok.

https://github.com/tytso/e2fsprogs/commit/53464654bd33e58e3fff079f34261b823d839f3b

Best regards,
Romain


>>
>> ---
>> For the stable branch 2021.08.x, I'll send a different patch, that just
>> backports the upstream fix.
> 
> In fact, 2021.08 still has e2fsprogs 1.46.2, which is not affected; so,
> no backports needed.
> 
> Regards,
> Yann E. MORIN.
> 
>> ---
>>  package/e2fsprogs/e2fsprogs.hash | 2 +-
>>  package/e2fsprogs/e2fsprogs.mk   | 2 +-
>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
>> index 3661801199..63c7c9929f 100644
>> --- a/package/e2fsprogs/e2fsprogs.hash
>> +++ b/package/e2fsprogs/e2fsprogs.hash
>> @@ -1,5 +1,5 @@
>>  # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
>> -sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
>> +sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
>>  # Locally calculated
>>  sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
>>  sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
>> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
>> index 5528ecbc34..ca891066a2 100644
>> --- a/package/e2fsprogs/e2fsprogs.mk
>> +++ b/package/e2fsprogs/e2fsprogs.mk
>> @@ -4,7 +4,7 @@
>>  #
>>  ################################################################################
>>  
>> -E2FSPROGS_VERSION = 1.46.3
>> +E2FSPROGS_VERSION = 1.46.4
>>  E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
>>  E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
>>  E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
>> -- 
>> 2.25.1
>>
> 

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-08 19:54   ` Romain Naour
@ 2021-09-08 20:22     ` Romain Naour
  2021-09-08 20:22     ` Romain Naour
  1 sibling, 0 replies; 7+ messages in thread
From: Romain Naour @ 2021-09-08 20:22 UTC (permalink / raw)
  To: Yann E. MORIN, buildroot; +Cc: Kevin Tomary, Leon de Rooij

Hello Yann, All,

Le 08/09/2021 à 21:54, Romain Naour a écrit :
> Hello Yann, All,
> 
> Le 05/09/2021 à 19:44, Yann E. MORIN a écrit :
>> All,
>>
>> On 2021-09-05 19:34 +0200, Yann E. MORIN spake thusly:
>>> Version 1.46.3 had a regression, which meant the file that would store
>>> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
>>> generate the filesystem:
>>>
>>>     mkfs.ext4: No such file or directory while trying to determine
>>>     filesystem size
>>>
>>> The regression was fixed upstream, and is now part of the 1.46.4
>>> release, so bump to that release.
>>>
>>> Fixes: #14196
>>>
>>> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
>>> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
>>> Reported-by: Leon de Rooij <leon@exquisip.nl>
> 
> There is another regression with e2fsprog 1.46.4:
> 
> https://gitlab.com/kubu93/buildroot/-/jobs/1568672940
> 
> With the fix backported to 1.46.3 the ext4 test is ok.
> 
> https://github.com/tytso/e2fsprogs/commit/53464654bd33e58e3fff079f34261b823d839f3b

Actually there is no regression, e2fsprogs deprecated filesystems with 128-byte
inodes. Now mkfs.ext4 use 256-byte inodes even for small filesystems.

https://github.com/tytso/e2fsprogs/commit/a23b50cdb55cb826b8745cbc37429c93f7b60c66

We have to update our test.

Best regards,
Romain


> 
> Best regards,
> Romain
> 
> 
>>>
>>> ---
>>> For the stable branch 2021.08.x, I'll send a different patch, that just
>>> backports the upstream fix.
>>
>> In fact, 2021.08 still has e2fsprogs 1.46.2, which is not affected; so,
>> no backports needed.
>>
>> Regards,
>> Yann E. MORIN.
>>
>>> ---
>>>  package/e2fsprogs/e2fsprogs.hash | 2 +-
>>>  package/e2fsprogs/e2fsprogs.mk   | 2 +-
>>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
>>> index 3661801199..63c7c9929f 100644
>>> --- a/package/e2fsprogs/e2fsprogs.hash
>>> +++ b/package/e2fsprogs/e2fsprogs.hash
>>> @@ -1,5 +1,5 @@
>>>  # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
>>> -sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
>>> +sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
>>>  # Locally calculated
>>>  sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
>>>  sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
>>> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
>>> index 5528ecbc34..ca891066a2 100644
>>> --- a/package/e2fsprogs/e2fsprogs.mk
>>> +++ b/package/e2fsprogs/e2fsprogs.mk
>>> @@ -4,7 +4,7 @@
>>>  #
>>>  ################################################################################
>>>  
>>> -E2FSPROGS_VERSION = 1.46.3
>>> +E2FSPROGS_VERSION = 1.46.4
>>>  E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
>>>  E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
>>>  E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
>>> -- 
>>> 2.25.1
>>>
>>
> 

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-08 19:54   ` Romain Naour
  2021-09-08 20:22     ` Romain Naour
@ 2021-09-08 20:22     ` Romain Naour
  2021-09-10 16:24       ` Yann E. MORIN
  1 sibling, 1 reply; 7+ messages in thread
From: Romain Naour @ 2021-09-08 20:22 UTC (permalink / raw)
  To: Yann E. MORIN, buildroot; +Cc: Kevin Tomary, Leon de Rooij

Hello Yann, All,

Le 08/09/2021 à 21:54, Romain Naour a écrit :
> Hello Yann, All,
> 
> Le 05/09/2021 à 19:44, Yann E. MORIN a écrit :
>> All,
>>
>> On 2021-09-05 19:34 +0200, Yann E. MORIN spake thusly:
>>> Version 1.46.3 had a regression, which meant the file that would store
>>> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
>>> generate the filesystem:
>>>
>>>     mkfs.ext4: No such file or directory while trying to determine
>>>     filesystem size
>>>
>>> The regression was fixed upstream, and is now part of the 1.46.4
>>> release, so bump to that release.
>>>
>>> Fixes: #14196
>>>
>>> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
>>> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
>>> Reported-by: Leon de Rooij <leon@exquisip.nl>
> 
> There is another regression with e2fsprog 1.46.4:
> 
> https://gitlab.com/kubu93/buildroot/-/jobs/1568672940
> 
> With the fix backported to 1.46.3 the ext4 test is ok.
> 
> https://github.com/tytso/e2fsprogs/commit/53464654bd33e58e3fff079f34261b823d839f3b

Actually there is no regression, e2fsprogs deprecated filesystems with 128-byte
inodes. Now mkfs.ext4 use 256-byte inodes even for small filesystems.

https://github.com/tytso/e2fsprogs/commit/a23b50cdb55cb826b8745cbc37429c93f7b60c66

We have to update our test.

Best regards,
Romain


> 
> Best regards,
> Romain
> 
> 
>>>
>>> ---
>>> For the stable branch 2021.08.x, I'll send a different patch, that just
>>> backports the upstream fix.
>>
>> In fact, 2021.08 still has e2fsprogs 1.46.2, which is not affected; so,
>> no backports needed.
>>
>> Regards,
>> Yann E. MORIN.
>>
>>> ---
>>>  package/e2fsprogs/e2fsprogs.hash | 2 +-
>>>  package/e2fsprogs/e2fsprogs.mk   | 2 +-
>>>  2 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
>>> index 3661801199..63c7c9929f 100644
>>> --- a/package/e2fsprogs/e2fsprogs.hash
>>> +++ b/package/e2fsprogs/e2fsprogs.hash
>>> @@ -1,5 +1,5 @@
>>>  # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
>>> -sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
>>> +sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
>>>  # Locally calculated
>>>  sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
>>>  sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
>>> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
>>> index 5528ecbc34..ca891066a2 100644
>>> --- a/package/e2fsprogs/e2fsprogs.mk
>>> +++ b/package/e2fsprogs/e2fsprogs.mk
>>> @@ -4,7 +4,7 @@
>>>  #
>>>  ################################################################################
>>>  
>>> -E2FSPROGS_VERSION = 1.46.3
>>> +E2FSPROGS_VERSION = 1.46.4
>>>  E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
>>>  E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
>>>  E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
>>> -- 
>>> 2.25.1
>>>
>>
> 

_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression
  2021-09-08 20:22     ` Romain Naour
@ 2021-09-10 16:24       ` Yann E. MORIN
  0 siblings, 0 replies; 7+ messages in thread
From: Yann E. MORIN @ 2021-09-10 16:24 UTC (permalink / raw)
  To: Romain Naour; +Cc: Kevin Tomary, Leon de Rooij, buildroot

Romain, All,

On 2021-09-08 22:22 +0200, Romain Naour spake thusly:
> Le 08/09/2021 à 21:54, Romain Naour a écrit :
> > Le 05/09/2021 à 19:44, Yann E. MORIN a écrit :
> >> On 2021-09-05 19:34 +0200, Yann E. MORIN spake thusly:
> >>> Version 1.46.3 had a regression, which meant the file that would store
> >>> the filesystem image had to pre-exist, or mkfs.ext2 would fail to
> >>> generate the filesystem:
> >>>
> >>>     mkfs.ext4: No such file or directory while trying to determine
> >>>     filesystem size
> >>>
> >>> The regression was fixed upstream, and is now part of the 1.46.4
> >>> release, so bump to that release.
> >>>
> >>> Fixes: #14196
> >>>
> >>> Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
> >>> Reported-by: Kevin Tomary <kevin.tomary@hotmail.com>
> >>> Reported-by: Leon de Rooij <leon@exquisip.nl>
> > 
> > There is another regression with e2fsprog 1.46.4:
> > 
> > https://gitlab.com/kubu93/buildroot/-/jobs/1568672940
> > 
> > With the fix backported to 1.46.3 the ext4 test is ok.
> > 
> > https://github.com/tytso/e2fsprogs/commit/53464654bd33e58e3fff079f34261b823d839f3b
> 
> Actually there is no regression, e2fsprogs deprecated filesystems with 128-byte
> inodes. Now mkfs.ext4 use 256-byte inodes even for small filesystems.
> https://github.com/tytso/e2fsprogs/commit/a23b50cdb55cb826b8745cbc37429c93f7b60c66

Oh, v ery good catch! I was banging my head on this a while ago, and was
pretty darn stuck... Thanks!

> We have to update our test.

The inode size change was part of 1.46.4, not 1.46.3, so it made sense
to fix the test at the same time we bump the version, which I've done
and pushed now. Thanks!

Regards,
Yann E. MORIN.

> Best regards,
> Romain
> 
> 
> > 
> > Best regards,
> > Romain
> > 
> > 
> >>>
> >>> ---
> >>> For the stable branch 2021.08.x, I'll send a different patch, that just
> >>> backports the upstream fix.
> >>
> >> In fact, 2021.08 still has e2fsprogs 1.46.2, which is not affected; so,
> >> no backports needed.
> >>
> >> Regards,
> >> Yann E. MORIN.
> >>
> >>> ---
> >>>  package/e2fsprogs/e2fsprogs.hash | 2 +-
> >>>  package/e2fsprogs/e2fsprogs.mk   | 2 +-
> >>>  2 files changed, 2 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/package/e2fsprogs/e2fsprogs.hash b/package/e2fsprogs/e2fsprogs.hash
> >>> index 3661801199..63c7c9929f 100644
> >>> --- a/package/e2fsprogs/e2fsprogs.hash
> >>> +++ b/package/e2fsprogs/e2fsprogs.hash
> >>> @@ -1,5 +1,5 @@
> >>>  # From https://mirrors.edge.kernel.org/pub/linux/kernel/people/tytso/e2fsprogs/v1.46.3/sha256sums.asc
> >>> -sha256  86d1580facdd49f2e0e6b027e26b1e6c48af538762dc40aeed2a87153c1f11b7  e2fsprogs-1.46.3.tar.xz
> >>> +sha256  b11042533c1b1dcf17512f0da48e05b0c573dada1dd8b762864d10f4dc399713  e2fsprogs-1.46.4.tar.xz
> >>>  # Locally calculated
> >>>  sha256  5da5ef153e559c1d990d4c3eedbedd4442db892d37eae1f35fff069de8ec9020  NOTICE
> >>>  sha256  032989b508f1a72ebee5b3417e55d06d473f9ee203e45ab11864a7e49cdec63d  lib/ss/mit-sipb-copyright.h
> >>> diff --git a/package/e2fsprogs/e2fsprogs.mk b/package/e2fsprogs/e2fsprogs.mk
> >>> index 5528ecbc34..ca891066a2 100644
> >>> --- a/package/e2fsprogs/e2fsprogs.mk
> >>> +++ b/package/e2fsprogs/e2fsprogs.mk
> >>> @@ -4,7 +4,7 @@
> >>>  #
> >>>  ################################################################################
> >>>  
> >>> -E2FSPROGS_VERSION = 1.46.3
> >>> +E2FSPROGS_VERSION = 1.46.4
> >>>  E2FSPROGS_SOURCE = e2fsprogs-$(E2FSPROGS_VERSION).tar.xz
> >>>  E2FSPROGS_SITE = $(BR2_KERNEL_MIRROR)/linux/kernel/people/tytso/e2fsprogs/v$(E2FSPROGS_VERSION)
> >>>  E2FSPROGS_LICENSE = GPL-2.0, MIT-like with advertising clause (libss and libet)
> >>> -- 
> >>> 2.25.1
> >>>
> >>
> > 
> 

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@lists.buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2021-09-10 16:25 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-05 17:34 [Buildroot] [PATCH] package/e2fsprogs: bump version to fix regression Yann E. MORIN
2021-09-05 17:44 ` Yann E. MORIN
2021-09-08 19:54   ` Romain Naour
2021-09-08 20:22     ` Romain Naour
2021-09-08 20:22     ` Romain Naour
2021-09-10 16:24       ` Yann E. MORIN
2021-09-06  9:07 ` Michael Walle

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.