Re: [PATCH] fix sys-process/cronie compilation under gentoo+musl

daggs <[email protected]>
Newsgroups gmane.linux.gentoo.hardened
Message-ID <trinity-90994d1b-503a-4296-82cf-0146237e0aa7-1416029047793@3capp-mailcom-bs15>
Greetings,

> Sent: Friday, November 14, 2014 at 11:05 PM
> From: "Tóth Attila" <[email protected]>
> To: [email protected]
> Subject: Re: [gentoo-hardened] [PATCH] fix sys-process/cronie compilation under gentoo+musl
>
> It's hard to spot the missing define in your patch, which includes a
> complete list of files of a whole package. Most of the material is
> consumed by the Changelog, tl;dr.
> 
> Do you mean these two?
> 
> +#include <limits.h>
> 
> +/* match gnu to posix */
> +#define MAXNAMLEN NAME_MAX
> 
> I would suggest to attach the responsible patch of the missing define only
> and the necessary diff of a given ebuild file. That two files could be
> enough, I guess.
> 
> The same suggestions goes for the previous offer.
> 
so the right way to do this is to add the fix patch, a patch that modifies an existing ebuild and if the ebuild is not present in the musl branch I should mentioned that?
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.