Re: PSA: cbindgen will be required to build in the near future.

2018-08-19 Thread Nika Layzell
That would be nice to do :-) - moving these larger generic build-time dependencies to being downloaded with bootstrap could really help improve build times! That being said, IIRC there is some interest in building pre-export/export build capability for other more-mozilla-specific/unstable

Re: PSA: cbindgen will be required to build in the near future.

2018-08-17 Thread Jan Beich
Emilio Cobos Álvarez writes: > Hi, > > In bug 1478813 (now on inbound), I've added a new toolchain job and > code to install cbindgen[1] via `mach bootstrap`, and it will become a > hard build dependency in the near future. > > We use it on the style system to generate C++ structs / enums from

Re: PSA: cbindgen will be required to build in the near future.

2018-08-17 Thread Emilio Cobos Álvarez
On 8/17/18 2:53 PM, Xidorn Quan wrote: I'm actually thinking about pulling bindgen out from build-dependency and make it use binary as well. bindgen was made a build-dependency and build in the process because at that time, bindgen wasn't stable enough, and we usually need to fix it for stylo

Re: PSA: cbindgen will be required to build in the near future.

2018-08-17 Thread Xidorn Quan
On Fri, Aug 17, 2018, at 10:19 PM, Emilio Cobos Álvarez wrote: > Adding support for general crates.io crates to be built this way and > appear somewhere on the objdir seemed way more build system hackery than > what I could do in a reasonable timeframe with my limited knowledge of > the build

Re: PSA: cbindgen will be required to build in the near future.

2018-08-17 Thread Emilio Cobos Álvarez
On 8/17/18 1:37 PM, Jan Beich wrote: Emilio Cobos Álvarez writes: Hi, In bug 1478813 (now on inbound), I've added a new toolchain job and code to install cbindgen[1] via `mach bootstrap`, and it will become a hard build dependency in the near future. We use it on the style system to

PSA: cbindgen will be required to build in the near future.

2018-08-15 Thread Emilio Cobos Álvarez
Hi, In bug 1478813 (now on inbound), I've added a new toolchain job and code to install cbindgen[1] via `mach bootstrap`, and it will become a hard build dependency in the near future. We use it on the style system to generate C++ structs / enums from the Rust definitions (as opposed to