[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [RFC PATCH SET] build system: update kconfig to bring new features and fixes
From:       Xabier Oneca  --  xOneca <xoneca () gmail ! com>
Date:       2022-04-26 22:02:33
Message-ID: CACkgH73nNz_yLuGDAupNPvYvOpaWBqwT5RtOE64D56XLF+ZXDA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Hi Ron,

It's probably worth pointing out the patches are nearly a year old.
> I started from BusyBox commit 972e29881, otherwise there were problems
> applying them.
>

Yeah! They are pretty old. I did forgot that I sent those patches :P

I just rebased the patches over the latest tree (fc7868602) and it merged
without conflicts too.


> Not to make it too easy for myself, though, I was working on the BusyBox
> port to Microsoft Windows which I maintain[1].  Naturally this has many
> tweaks to the build system.
>

That's a hard test for this patchset! :)

The first three patches went in just fine.


I did expect so.


> The last two required careful
> shepherding


Expected too. Sorry. Could not find a better way to post these last two
patches


> but got there in the end.  Cross compiling for Windows on
> Linux worked fine, as did a POSIX build on Linux.


That's good news!


> I haven't tried a
> native compilation on Windows yet.
>
> On the whole, this was less troublesome than I expected.
>

 I appreciate your effort for review this patchset. If it gets merged, I
have a couple more tangential improvements in the backlog to post...

Cheers,

Xabier Oneca_,,_

[Attachment #5 (text/html)]

<div dir="ltr"><div>Hi Ron,<br></div><div class="gmail_quote"><div dir="ltr" \
class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px \
0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">It&#39;s probably \
worth pointing out the patches are nearly a year old.<br> I started from BusyBox \
commit 972e29881, otherwise there were problems<br> applying \
them.<br></blockquote><div><br></div><div>Yeah! They are pretty old. I did forgot \
that I sent those patches :P<br><br>I just  rebased the patches over the latest tree \
(fc7868602) and it merged without conflicts too.<br></div><div>  </div><blockquote \
class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">

Not to make it too easy for myself, though, I was working on the BusyBox<br>
port to Microsoft Windows which I maintain[1].   Naturally this has many<br>
tweaks to the build system.<br></blockquote><div><br></div><div>That&#39;s a hard \
test for this patchset! :)<br><br></div><blockquote class="gmail_quote" \
style="margin:0px 0px 0px 0.8ex;border-left:1px solid \
rgb(204,204,204);padding-left:1ex">

The first three patches went in just fine.</blockquote><div><br></div><div>I did \
expect so.<br></div><div>  </div><blockquote class="gmail_quote" style="margin:0px \
0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The last two \
required careful<br> shepherding</blockquote><div><br></div><div>Expected too. Sorry. \
Could not find a better way to post these last two patches<br></div><div>  \
</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px \
solid rgb(204,204,204);padding-left:1ex">but got there in the end.   Cross compiling \
for Windows on<br> Linux worked fine, as did a POSIX build on \
Linux.</blockquote><div><br></div><div>That&#39;s good news!<br></div><div>  \
</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px \
solid rgb(204,204,204);padding-left:1ex">I haven&#39;t tried a<br> native compilation \
on Windows yet.<br> <br>
On the whole, this was less troublesome than I expected.<br>
</blockquote><div><br></div><div>  I appreciate your effort for review this patchset. \
If it gets merged, I have a couple more tangential improvements in the backlog to \
post...<br><br></div><div>Cheers,<br><br></div><div>Xabier \
Oneca_,,_<br></div></div></div>



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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic