[Buildroot] [autobuild.buildroot.net] Build results for 2015-11-22

[Vicente Olivert Riera <Vincent.Riera@imgtec.com>]

Hi Romain,

On 23/11/15 21:09, Romain Naour wrote:
> Hello,
> 
>>       x86_64 |                  lvm2-2.02.132 | NOK | http://autobuild.buildroot.net/results/964b5909dbac97d56b87e22e5141d62c54eba893/
> 
> commands/toolcontext.c: In function 'create_toolcontext':
> commands/toolcontext.c:1795:10: error: assignment of read-only variable 'stdin'
>     stdin = new_stream;
> 
> This is a musl issue and generally fixed by using freopen(). See [1] [2].
> 
> There is a patch from alpinelinux [3] but I'm wondering if it could be simpler
> to get ride of _reopen_stream() an use freopen() instead.
> 
> Vincente, what do you think ?

you told me in the IRC that this patch could affect other C libraries,
so, my opinion is that it would be better to work with upstream and fix
the problem there, and then backport the fix into Buildroot.

Regards,

Vincent.

> Best regards,
> Romain
> 
> [1]
> https://github.com/doughdemon/fdupes/commit/fe2d8334bfa28ff05fa1a2ffe8555a19b41be0ea
> [2] https://github.com/adrianlopezroche/fdupes/pull/31
> [3]
> http://git.alpinelinux.org/cgit/aports/tree/main/lvm2/fix-stdio-usage.patch?h=v3.2.3
>