[Buildroot] [PATCH 1/1] package/macchanger: fix musl build

Jörg Krause joerg.krause at embedded.rocks
Fri May 6 06:26:58 UTC 2016


Dear Arnout,

On Mo, 2016-05-02 at 23:32 +0200, Arnout Vandecappelle wrote:
> On 05/02/16 21:40, Jörg Krause wrote:
> > Hi,
> > 
> > On Sa, 2016-01-23 at 23:43 +0100, Bernd Kuhls wrote:
> [snip]
> > > +@@ -113,7 +113,7 @@ mc_net_info_get_permanent_mac (const net
> > > + 	epa->size = IFHWADDRLEN;
> > > +
> > > + 	memcpy(&req, &(net->dev), sizeof(struct ifreq));
> > > +-	req.ifr_data = (caddr_t)epa;
> > > ++	req.ifr_data = (char *)epa;
> > > +
> > > + 	if (ioctl(net->sock, SIOCETHTOOL, &req) < 0) {
> > > + 		perror ("[ERROR] Could not read permanent
> > > MAC");
> > 
> > any reason why this patch is marked as "Changes Requested"? The
> > build
> > error is still present...
> 
>   IIRC, Bernd posted almost a hundred patches at the time for musl
> fixes, most 
> of which were taken from Alpine Linux, and almost all of which had
> insufficient 
> explanation of the problem and of the fix (nothing more than "fix
> musl build" 
> and a reference to the alpine patch). Also many of them were simply
> incorrect: 
> they would maybe fix the build, but possibly introducing other bugs
> or sometimes 
> just breaking the code. So after reviewing and rejecting a dozen of
> them, 
> ThomasP just made a generic comment that all of them should be done
> more 
> carefully and marked all of them as Changes Requested.
> 
>   If you would like to recover and review them, just select in
> patchwork patches 
> from Bernd that are marked as changes requested and that have musl in
> the 
> subject. Feel free to repost (with better commit messages of course).

I see! Yes, I will do this for some packages.

>   This one specifically does look good to me - except for the commit
> message, 
> which should be something like:
> 
> caddr_t is a BSD type. POSIX usually uses void* instead, but
> specifically in 
> struct ifreq the type of ifr_data is char*.
> 

Many thanks!

Best regards
Jörg Krause


More information about the buildroot mailing list