Here is a fresh patch, all comments welcome! As suggested, I've put all the changes we need into the configure.ac file instead of just the first one. Again, this patch is to add the NaCl operating system for chromium into the emacs configure.ac file. As I mentioned in previous patch requests, I'm adding support for a new build target, the chromium browser's Native Client (NaCl) platform. We have a patch for emacs working today (for some definitions of working ^_^) in NaCl ports, and we would like to port the changes back to the emacs code base. Our work in progress is visible in the "naclports" github project. On Wed, Mar 18, 2015 at 9:04 AM, Pete Williamson wrote: > Since there has been no other feedback on this topic, I'll take Michal's > suggestion (and clear up Paul's confusion), and make all the changes to the > config.ac file at once, stay tuned for an updated patch. > > On Fri, Mar 13, 2015 at 5:29 PM, Pete Williamson > wrote: > >> The actual patch is to configure.ac. The diff is showing things that I >> would move from my current "configure" file to "configure.ac" >> >> On Fri, Mar 13, 2015 at 5:28 PM, Paul Eggert wrote: >> >>> Pete Williamson wrote: >>> >>>> --- a/configure >>>> +++ b/configure >>>> >>> >>> 'configure' is generated automatically from 'configure.ac' and other >>> files, so patches to 'configure' are ineffective. You need to patch >>> configure.ac and the other files instead. >>> >>> If you need to patch config.guess and/or config.sub, they come from a >>> different project and those changes need to be sent upstream, as described >>> in those files. >>> >> >> >