[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

All,

Here is session 6 of the patchwork cleanup. For those who haven't
followed previous sessions, here is a quick recap: every session we'll
be posting the list of the 10 oldest patches, with the original patch
contributors in copy. During two weeks, we have the opportunity to
discuss the patch, whether it is still necessary or not, how it should
be changed to be merged.

In particular, I'm interested in knowing if the original contributor is
still interested by the patch, or if someone else is interested in
adopting the patch and sending an updated version, or if the patch no
longer makes sense due to other changes made in Buildroot.

All patches that have not received any comment or attention before the
deadline will be removed from patchwork. This is aggressive, but we have
many 'dead' patches sitting in patchwork, and this cannot work.

Deadline for this session is February 2 (FOSDEM + Buildroot developer days!).

Below are the 11 oldest patches (obtained with a slightly modified
version of pwclient):
[ pwclient list -s New -n 21 ] (and skip Delegated patches)

(Note: temporarily increased amount to 11 to include the entire python
patch series)


Update dmraid from 1.0.0-rc15 to 1.0.0-rc16-3.
Antony Vennard <arv@vx9.co.uk>
http://patchwork.ozlabs.org/patch/213230

Linux: enable local Linux tarball in config
Stephan Hoffmann <sho@relinux.de>
http://patchwork.ozlabs.org/patch/217204

minimal dockstar setup
Steve Calfee <stevecalfee@gmail.com>
http://patchwork.ozlabs.org/patch/217639

scripts: xorg-release: handle case when version needs downgrade
Nicolas Dechesne <ndec13@gmail.com>
http://patchwork.ozlabs.org/patch/218109

[v2] configs: at91sam9260eknf: new configuration for Atmel AT91SAM9260-EK
Fabio Porcedda <fabio.porcedda@gmail.com>
http://patchwork.ozlabs.org/patch/218928

[1/1] python: fix to ensure libpython is stripped
Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
http://patchwork.ozlabs.org/patch/220024

[v4,1/5] Python menu reorganization
kpa_info at yahoo.fr
http://patchwork.ozlabs.org/patch/220706

[v4,2/5] Add python3 config directory simlink
kpa_info at yahoo.fr
http://patchwork.ozlabs.org/patch/220707

[v4,3/5] Add PYTHON variable to ease the support of both version
kpa_info at yahoo.fr
http://patchwork.ozlabs.org/patch/220705

[v4,4/5] python-bottle: Convert package to be used with python2 or python3
kpa_info at yahoo.fr
http://patchwork.ozlabs.org/patch/220708

[v4,5/5] python-serial: Convert package to be used with python2 or python3
kpa_info at yahoo.fr
http://patchwork.ozlabs.org/patch/220709


Thanks in advance for your feedback!

Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Przemyslaw Wrzos]

Hello Thomas,

On Tue, 21 Jan 2014 at 08:06:25, Thomas De Schampheleire wrote:

> In particular, I'm interested in knowing if the original contributor is
> still interested by the patch, or if someone else is interested in
> adopting the patch and sending an updated version, or if the patch no
> longer makes sense due to other changes made in Buildroot.

...

> [1/1] python: fix to ensure libpython is stripped Przemyslaw Wrzos
> <przemyslaw.wrzos@calyptech.com>
http://patchwork.ozlabs.org/patch/220024

I'd still like to see this one included in some form. For embedded systems
that count their image sizes in the tens of megabytes, this patch can make
a not insignificant difference.

The patch still applies as is to the latest source tree. Let me know if
anything else need to be done from my end.

Cheers,
Przemyslaw Wrzos

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

Hi Przemyslaw,

On Tue, Jan 21, 2014 at 3:53 AM, Przemyslaw Wrzos
<przemyslaw.wrzos@calyptech.com> wrote:
> Hello Thomas,
>
> On Tue, 21 Jan 2014 at 08:06:25, Thomas De Schampheleire wrote:
>
>> In particular, I'm interested in knowing if the original contributor is
>> still interested by the patch, or if someone else is interested in
>> adopting the patch and sending an updated version, or if the patch no
>> longer makes sense due to other changes made in Buildroot.
>
> ...
>
>> [1/1] python: fix to ensure libpython is stripped Przemyslaw Wrzos
>> <przemyslaw.wrzos@calyptech.com>
> http://patchwork.ozlabs.org/patch/220024
>
> I'd still like to see this one included in some form. For embedded systems
> that count their image sizes in the tens of megabytes, this patch can make
> a not insignificant difference.
>
> The patch still applies as is to the latest source tree. Let me know if
> anything else need to be done from my end.

If you have time to resubmit your patch to the list, I think that
would be best. It will be back at the top of the list, encouraging
review and eventually integration. You could add a note below the
'---' line stating that this patch was part of patchwork cleanup #6.

Thanks!
Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Stephan Hoffmann]

Am 20.01.2014 22:06, schrieb Thomas De Schampheleire:
> All,
>
> Here is session 6 of the patchwork cleanup. For those who haven't
> followed previous sessions, here is a quick recap: every session we'll
> be posting the list of the 10 oldest patches, with the original patch
> contributors in copy. During two weeks, we have the opportunity to
> discuss the patch, whether it is still necessary or not, how it should
> be changed to be merged.
>
> In particular, I'm interested in knowing if the original contributor is
> still interested by the patch, or if someone else is interested in
> adopting the patch and sending an updated version, or if the patch no
> longer makes sense due to other changes made in Buildroot.
.....
> ...
>
> Linux: enable local Linux tarball in config
> Stephan Hoffmann <sho@relinux.de>
> http://patchwork.ozlabs.org/patch/217204
Thomas, all,

I still think that this would be a useful feature for buildroot,
especially when dealing with soc or boards that come with a kernel
tarball from the vendor that is not easily accessible online.

I made this patch because I had this situation in one of my projects. If
there is interest in committing it I think that I can spare some time to
rebase it to the current HEAD.

Kind regards

Stephan
>
>
> Thanks in advance for your feedback!
>
> Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

Hi Stephan,

On Tue, Jan 21, 2014 at 8:58 AM, Stephan Hoffmann <sho@relinux.de> wrote:
> Am 20.01.2014 22:06, schrieb Thomas De Schampheleire:
>> All,
>>
>> Here is session 6 of the patchwork cleanup. For those who haven't
>> followed previous sessions, here is a quick recap: every session we'll
>> be posting the list of the 10 oldest patches, with the original patch
>> contributors in copy. During two weeks, we have the opportunity to
>> discuss the patch, whether it is still necessary or not, how it should
>> be changed to be merged.
>>
>> In particular, I'm interested in knowing if the original contributor is
>> still interested by the patch, or if someone else is interested in
>> adopting the patch and sending an updated version, or if the patch no
>> longer makes sense due to other changes made in Buildroot.
> .....
>> ...
>>
>> Linux: enable local Linux tarball in config
>> Stephan Hoffmann <sho@relinux.de>
>> http://patchwork.ozlabs.org/patch/217204
> Thomas, all,
>
> I still think that this would be a useful feature for buildroot,
> especially when dealing with soc or boards that come with a kernel
> tarball from the vendor that is not easily accessible online.
>

I just looked at your patch, but it is not needed to achieve what you
want. If you specify:
file:///path/to/local/file
then the site method will be automatically set to file (just tested).
Could you verify that this indeed fits your use case?

Thanks,
Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Fabio Porcedda]

On Mon, Jan 20, 2014 at 10:06 PM, Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
<snip>
> [v2] configs: at91sam9260eknf: new configuration for Atmel AT91SAM9260-EK
> Fabio Porcedda <fabio.porcedda@gmail.com>
> http://patchwork.ozlabs.org/patch/218928
<snip>

This patch is superseded by an updated patch that has already been merged.

Regards
-- 
Fabio Porcedda

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

Hi Fabio,

On Tue, Jan 21, 2014 at 10:02 AM, Fabio Porcedda
<fabio.porcedda@gmail.com> wrote:
> On Mon, Jan 20, 2014 at 10:06 PM, Thomas De Schampheleire
> <patrickdepinguin@gmail.com> wrote:
> <snip>
>> [v2] configs: at91sam9260eknf: new configuration for Atmel AT91SAM9260-EK
>> Fabio Porcedda <fabio.porcedda@gmail.com>
>> http://patchwork.ozlabs.org/patch/218928
> <snip>
>
> This patch is superseded by an updated patch that has already been merged.
>

Thanks! Marked as such in patchwork...

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Stephan Hoffmann]

On 21.01.2014 09:18, Thomas De Schampheleire wrote:
> Hi Stephan,
>
> On Tue, Jan 21, 2014 at 8:58 AM, Stephan Hoffmann <sho@relinux.de> wrote:
>> Am 20.01.2014 22:06, schrieb Thomas De Schampheleire:
>>> All,
>>>
>>> Here is session 6 of the patchwork cleanup. For those who haven't
>>> followed previous sessions, here is a quick recap: every session we'll
>>> be posting the list of the 10 oldest patches, with the original patch
>>> contributors in copy. During two weeks, we have the opportunity to
>>> discuss the patch, whether it is still necessary or not, how it should
>>> be changed to be merged.
>>>
>>> In particular, I'm interested in knowing if the original contributor is
>>> still interested by the patch, or if someone else is interested in
>>> adopting the patch and sending an updated version, or if the patch no
>>> longer makes sense due to other changes made in Buildroot.
>> .....
>>> ...
>>>
>>> Linux: enable local Linux tarball in config
>>> Stephan Hoffmann <sho@relinux.de>
>>> http://patchwork.ozlabs.org/patch/217204
>> Thomas, all,
>>
>> I still think that this would be a useful feature for buildroot,
>> especially when dealing with soc or boards that come with a kernel
>> tarball from the vendor that is not easily accessible online.
>>
> I just looked at your patch, but it is not needed to achieve what you
> want. If you specify:
> file:///path/to/local/file
> then the site method will be automatically set to file (just tested).
> Could you verify that this indeed fits your use case?
Yes, it does, and it does with the old buildroot I had then, too. So my
patch is obsolete, please remove it. I do not remember why I did not
find this last year.

Thanks

Stephan
>
> Thanks,
> Thomas


-- 
reLinux     -    Stephan Hoffmann
Am Schmidtgrund 124    50765 K?ln
Tel. +49.221.95595-19    Fax: -64
www.reLinux.de Stephan at reLinux.de

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

On Tue, Jan 21, 2014 at 10:39 AM, Stephan Hoffmann <sho@relinux.de> wrote:
[..]
>>>> Linux: enable local Linux tarball in config
>>>> Stephan Hoffmann <sho@relinux.de>
>>>> http://patchwork.ozlabs.org/patch/217204
>>> Thomas, all,
>>>
>>> I still think that this would be a useful feature for buildroot,
>>> especially when dealing with soc or boards that come with a kernel
>>> tarball from the vendor that is not easily accessible online.
>>>
>> I just looked at your patch, but it is not needed to achieve what you
>> want. If you specify:
>> file:///path/to/local/file
>> then the site method will be automatically set to file (just tested).
>> Could you verify that this indeed fits your use case?
> Yes, it does, and it does with the old buildroot I had then, too. So my
> patch is obsolete, please remove it. I do not remember why I did not
> find this last year.

Ok, great that we sorted this out. I'll mark the patch as rejected in patchwork.

Thanks,
Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Steve Calfee]

On Mon, Jan 20, 2014 at 1:06 PM, Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
> All,
>
> Here is session 6 of the patchwork cleanup. For those who haven't
> followed previous sessions, here is a quick recap: every session we'll
> be posting the list of the 10 oldest patches, with the original patch
> contributors in copy. During two weeks, we have the opportunity to
> discuss the patch, whether it is still necessary or not, how it should
> be changed to be merged.
>
> In particular, I'm interested in knowing if the original contributor is
> still interested by the patch, or if someone else is interested in
> adopting the patch and sending an updated version, or if the patch no
> longer makes sense due to other changes made in Buildroot.
>
> All patches that have not received any comment or attention before the
> deadline will be removed from patchwork. This is aggressive, but we have
> many 'dead' patches sitting in patchwork, and this cannot work.
>
> Deadline for this session is February 2 (FOSDEM + Buildroot developer days!).
>
<snip>
> minimal dockstar setup
> Steve Calfee <stevecalfee@gmail.com>
> http://patchwork.ozlabs.org/patch/217639
>

I guess you might as well drop the patch. At the time before rpi, the
$30 retail dockstar was the cheapest linux computer you could buy. I
thought others might be interested, but have received no comments, so
don't bother with this.

It still builds and works fine, but with later kernels beyond 3.6.8 (I
think) they changed the kernel for device tree, and that broke my
uboot. Changing uboot is a very dangerous thing, risking bricking the
dockstar, so I have stopped at that kernel for now. Still a nice NAS
for home, though.

Regards, Steve

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Przemyslaw Wrzos]

Hello Thomas,

On Tue, 21 Jan 2014 at 18:46:04, Thomas De Schampheleire wrote:

> If you have time to resubmit your patch to the list, I think that
> would be best. It will be back at the top of the list, encouraging
> review and eventually integration. You could add a note below the
> '---' line stating that this patch was part of patchwork cleanup #6.
> 
> Thanks!
> Thomas

No problem, I've resubmitted the patch (also CC'ed you on that e-mail).

Cheers,
Przemyslaw Wrzos

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

Hi Steve,

On Wed, Jan 22, 2014 at 11:10 PM, Steve Calfee <stevecalfee@gmail.com> wrote:
> On Mon, Jan 20, 2014 at 1:06 PM, Thomas De Schampheleire
> <patrickdepinguin@gmail.com> wrote:
>> All,
>>
>> Here is session 6 of the patchwork cleanup. For those who haven't
>> followed previous sessions, here is a quick recap: every session we'll
>> be posting the list of the 10 oldest patches, with the original patch
>> contributors in copy. During two weeks, we have the opportunity to
>> discuss the patch, whether it is still necessary or not, how it should
>> be changed to be merged.
>>
>> In particular, I'm interested in knowing if the original contributor is
>> still interested by the patch, or if someone else is interested in
>> adopting the patch and sending an updated version, or if the patch no
>> longer makes sense due to other changes made in Buildroot.
>>
>> All patches that have not received any comment or attention before the
>> deadline will be removed from patchwork. This is aggressive, but we have
>> many 'dead' patches sitting in patchwork, and this cannot work.
>>
>> Deadline for this session is February 2 (FOSDEM + Buildroot developer days!).
>>
> <snip>
>> minimal dockstar setup
>> Steve Calfee <stevecalfee@gmail.com>
>> http://patchwork.ozlabs.org/patch/217639
>>
>
> I guess you might as well drop the patch. At the time before rpi, the
> $30 retail dockstar was the cheapest linux computer you could buy. I
> thought others might be interested, but have received no comments, so
> don't bother with this.
>
> It still builds and works fine, but with later kernels beyond 3.6.8 (I
> think) they changed the kernel for device tree, and that broke my
> uboot. Changing uboot is a very dangerous thing, risking bricking the
> dockstar, so I have stopped at that kernel for now. Still a nice NAS
> for home, though.
>

Ok, thanks for the feedback.
I'll mark it as such in patchwork...

Best regards,
Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

Hi all,


On Mon, Jan 20, 2014 at 10:06 PM, Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
> All,
>
> Here is session 6 of the patchwork cleanup. For those who haven't
> followed previous sessions, here is a quick recap: every session we'll
> be posting the list of the 10 oldest patches, with the original patch
> contributors in copy. During two weeks, we have the opportunity to
> discuss the patch, whether it is still necessary or not, how it should
> be changed to be merged.
>
> In particular, I'm interested in knowing if the original contributor is
> still interested by the patch, or if someone else is interested in
> adopting the patch and sending an updated version, or if the patch no
> longer makes sense due to other changes made in Buildroot.
>
> All patches that have not received any comment or attention before the
> deadline will be removed from patchwork. This is aggressive, but we have
> many 'dead' patches sitting in patchwork, and this cannot work.
>
> Deadline for this session is February 2 (FOSDEM + Buildroot developer days!).
>
> Below are the 11 oldest patches (obtained with a slightly modified
> version of pwclient):
> [ pwclient list -s New -n 21 ] (and skip Delegated patches)
>
> (Note: temporarily increased amount to 11 to include the entire python
> patch series)

Here is an intermediate overview:

>
>
> Update dmraid from 1.0.0-rc15 to 1.0.0-rc16-3.
> Antony Vennard <arv@vx9.co.uk>
> http://patchwork.ozlabs.org/patch/213230

No feedback yet.

>
> Linux: enable local Linux tarball in config
> Stephan Hoffmann <sho@relinux.de>
> http://patchwork.ozlabs.org/patch/217204

Patch not needed, problem can be solved by prefixing URL with file://.

>
> minimal dockstar setup
> Steve Calfee <stevecalfee@gmail.com>
> http://patchwork.ozlabs.org/patch/217639

Rejected as per feedback of Steve.

>
> scripts: xorg-release: handle case when version needs downgrade
> Nicolas Dechesne <ndec13@gmail.com>
> http://patchwork.ozlabs.org/patch/218109

No feedback yet.

>
> [v2] configs: at91sam9260eknf: new configuration for Atmel AT91SAM9260-EK
> Fabio Porcedda <fabio.porcedda@gmail.com>
> http://patchwork.ozlabs.org/patch/218928

Superseded patch.

>
> [1/1] python: fix to ensure libpython is stripped
> Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
> http://patchwork.ozlabs.org/patch/220024

Resubmitted and thus superseded.

>
> [v4,1/5] Python menu reorganization
> kpa_info at yahoo.fr
> http://patchwork.ozlabs.org/patch/220706
>
> [v4,2/5] Add python3 config directory simlink
> kpa_info at yahoo.fr
> http://patchwork.ozlabs.org/patch/220707
>
> [v4,3/5] Add PYTHON variable to ease the support of both version
> kpa_info at yahoo.fr
> http://patchwork.ozlabs.org/patch/220705
>
> [v4,4/5] python-bottle: Convert package to be used with python2 or python3
> kpa_info at yahoo.fr
> http://patchwork.ozlabs.org/patch/220708
>
> [v4,5/5] python-serial: Convert package to be used with python2 or python3
> kpa_info at yahoo.fr
> http://patchwork.ozlabs.org/patch/220709

No feedback on the above yet.

Further feedback will be very much appreciated!

Thanks,
Thomas

[Buildroot] Patchwork oldest patches cleanup #6 (deadline February 2)

[Thomas De Schampheleire]

All,

On Mon, Jan 27, 2014 at 9:07 PM, Thomas De Schampheleire
<patrickdepinguin@gmail.com> wrote:
[..]
>>
>> Here is session 6 of the patchwork cleanup. For those who haven't
>> followed previous sessions, here is a quick recap: every session we'll
>> be posting the list of the 10 oldest patches, with the original patch
>> contributors in copy. During two weeks, we have the opportunity to
>> discuss the patch, whether it is still necessary or not, how it should
>> be changed to be merged.
>>
>> In particular, I'm interested in knowing if the original contributor is
>> still interested by the patch, or if someone else is interested in
>> adopting the patch and sending an updated version, or if the patch no
>> longer makes sense due to other changes made in Buildroot.
>>
>> All patches that have not received any comment or attention before the
>> deadline will be removed from patchwork. This is aggressive, but we have
>> many 'dead' patches sitting in patchwork, and this cannot work.
>>
>> Deadline for this session is February 2 (FOSDEM + Buildroot developer days!).
>>
>> Below are the 11 oldest patches (obtained with a slightly modified
>> version of pwclient):
>> [ pwclient list -s New -n 21 ] (and skip Delegated patches)
>>
>> (Note: temporarily increased amount to 11 to include the entire python
>> patch series)
>
> Here is an intermediate overview:

Here is the final conclusion of this cleanup session:

>
>>
>>
>> Update dmraid from 1.0.0-rc15 to 1.0.0-rc16-3.
>> Antony Vennard <arv@vx9.co.uk>
>> http://patchwork.ozlabs.org/patch/213230
>
> No feedback yet.

No feedback, but I'm keeping this patch as the version bump is
supposedly fixing a problem.
Adopters of this patch are welcome!

>
>>
>> Linux: enable local Linux tarball in config
>> Stephan Hoffmann <sho@relinux.de>
>> http://patchwork.ozlabs.org/patch/217204
>
> Patch not needed, problem can be solved by prefixing URL with file://.
>
>>
>> minimal dockstar setup
>> Steve Calfee <stevecalfee@gmail.com>
>> http://patchwork.ozlabs.org/patch/217639
>
> Rejected as per feedback of Steve.
>
>>
>> scripts: xorg-release: handle case when version needs downgrade
>> Nicolas Dechesne <ndec13@gmail.com>
>> http://patchwork.ozlabs.org/patch/218109
>
> No feedback yet.

No feedback, we'll discuss this on the developer days today or tomorrow...

>
>>
>> [v2] configs: at91sam9260eknf: new configuration for Atmel AT91SAM9260-EK
>> Fabio Porcedda <fabio.porcedda@gmail.com>
>> http://patchwork.ozlabs.org/patch/218928
>
> Superseded patch.
>
>>
>> [1/1] python: fix to ensure libpython is stripped
>> Przemyslaw Wrzos <przemyslaw.wrzos@calyptech.com>
>> http://patchwork.ozlabs.org/patch/220024
>
> Resubmitted and thus superseded.
>
>>
>> [v4,1/5] Python menu reorganization
>> kpa_info at yahoo.fr
>> http://patchwork.ozlabs.org/patch/220706
>>
>> [v4,2/5] Add python3 config directory simlink
>> kpa_info at yahoo.fr
>> http://patchwork.ozlabs.org/patch/220707
>>
>> [v4,3/5] Add PYTHON variable to ease the support of both version
>> kpa_info at yahoo.fr
>> http://patchwork.ozlabs.org/patch/220705
>>
>> [v4,4/5] python-bottle: Convert package to be used with python2 or python3
>> kpa_info at yahoo.fr
>> http://patchwork.ozlabs.org/patch/220708
>>
>> [v4,5/5] python-serial: Convert package to be used with python2 or python3
>> kpa_info at yahoo.fr
>> http://patchwork.ozlabs.org/patch/220709
>
> No feedback on the above yet.

No feedback, but I don't think we want to throw these patches away. In
the meantime I'll mark them as delegated to me. Hopefully we can
discuss this on the buildroot developer days too.

Thanks all!

Best regards,
Thomas