Re: Is GNU Make Accepting New Contributions

Noah Goldstein <[email protected]>
Newsgroups gmane.comp.gnu.make.general
Message-ID <CAFUsyfKKux7qJzO9V0Myh7sbRJ5yeRrXFz+VGsfE6m0CMgYgyg@mail.gmail.com>
On Wed, Apr 20, 2022 at 3:28 PM Noah Goldstein <[email protected]> wrote:
>
> On Wed, Apr 20, 2022 at 3:16 PM Paul Smith <[email protected]> wrote:
> >
> > On Wed, 2022-04-20 at 10:30 -0500, Noah Goldstein wrote:
> > > Is this project still accepting patches from new contributors? If so,
> > > what is the review process?
> > >
> > > I have my FSF agreement signed.
> >
> > We do accept changes from anyone.  However, we do not accept all
> > changes.  Bug fixes are always welcome.  GNU make is a critical
> > infrastructure project with a LOT of portability issues, dependency
> > issues, etc. so not all ideas for improvements/new features will be
> > accepted.
>
> Hi Paul,
>
> The change I am looking at are 2-fold.
>
>
> 1) Replace unnecessary calls to strcmp with memcmp (i.e cases
> where string length has already been compared)
>
> 2) Remove the `isupper()` branch in STRINGI_HASH. This is just
> objectively slower.
>
> The motivation is I'm seeing pretty high CPU usage in hash_find_slot
> and `strcmp` when making the GLIBC benchmark suite.

If those changes would be okay where should I send my patches?
>
> >
> > If you are planning on working on a larger change that may have
> > backward-compatibility, etc. implications it's best to discuss your
> > idea first, on the [email protected] mailing list.
> >
> > If you have a patch for a fix, you can email it (as an attachment
> > please) to the [email protected] mailing list.
> >
> > You can also file a bug (even if the change is not technically a bug
> > fix) then attach the patch to it on Savannah:
> > https://savannah.gnu.org/projects/make/
> >
> > Cheers!
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.