#general
- f
fast-nail-55400
06/26/2017, 9:27 PMbut alas not today - w
witty-crayon-22786
06/26/2017, 9:41 PM@fast-nail-55400 : https://github.com/pantsbuild/pants/pull/4706 - f
fast-nail-55400
06/26/2017, 10:54 PMI'll try the new version of that doc - f
fast-nail-55400
06/26/2017, 10:57 PMthe main difference is that instead of a
, the underling source target is ajava_libraryscalac_plugin - b
bright-apartment-76721
06/26/2017, 11:06 PMany reason y'all use crate-type = dylib instead of crate-type = cdylib? - w
witty-crayon-22786
06/26/2017, 11:07 PMneeds to be loaded by python - b
bright-apartment-76721
06/26/2017, 11:08 PMfrom #rust, they told me "you almost never want dylib unless you're writing a rust plugin; cdylib is what you usually want if you want a dylib for linking against typical C ABIs" - b
bright-apartment-76721
06/26/2017, 11:08 PMparaphrased 🙂 - b
bright-apartment-76721
06/26/2017, 11:08 PMI know nothing - b
bright-apartment-76721
06/26/2017, 11:08 PMI was just asking them about it and noticed that y'all were using dylib instead of cdylib - w
witty-crayon-22786
06/26/2017, 11:09 PMhm. yea, no idea then. - b
bright-apartment-76721
06/26/2017, 11:10 PMseems kind of counterintuitive that the one with the familiar name is the one you don't want 🤔 - b
bright-apartment-76721
06/26/2017, 11:10 PManyway, I was getting a super obscure build error in the pants engine with dylib - b
bright-apartment-76721
06/26/2017, 11:10 PMthat magically went away with cdylib - r
rough-minister-58256
06/26/2017, 11:12 PMI don’t think
existed when the engine was first writtencdylib - b
bright-apartment-76721
06/26/2017, 11:12 PMaha - b
bright-apartment-76721
06/26/2017, 11:12 PMmaybe that's it - b
bright-apartment-76721
06/26/2017, 11:12 PManyway, I'm trying to learn more about the distinction - b
bright-apartment-76721
06/26/2017, 11:13 PMapparently dylib includes a bunch of rust metadata that helps rust link against it - b
bright-apartment-76721
06/26/2017, 11:13 PMthe metadata also seemed to be the thing crashing my rustc, but it's hard to pin it down - r
rough-minister-58256
06/26/2017, 11:14 PMyeah looks like a size optimization - r
rough-minister-58256
06/26/2017, 11:15 PMcould probably switch us over to it and it’d hopefully just work - b
bright-apartment-76721
06/26/2017, 11:15 PMsomeone on #rust was saying "Rust dylibs are just terrible all around" 😛 - b
bright-apartment-76721
06/26/2017, 11:15 PMI can make a quick PR if you'd like - r
rough-minister-58256
06/26/2017, 11:15 PMsure, go for it - b
bright-apartment-76721
06/26/2017, 11:15 PMat the same time, it makes me uncomfortable to not understand weird crashes, but I've already invested way too much time in tracking this down - b
bright-apartment-76721
06/26/2017, 11:15 PMbut I'll do the PR - r
rough-minister-58256
06/26/2017, 11:15 PMyou’re seeing crashes compiling the engine? - r
rough-minister-58256
06/26/2017, 11:16 PMthat go away with
?s/dylib/cdylib/ - b
bright-apartment-76721
06/26/2017, 11:18 PMyeah, really weird