ck_ ( @ck_@discuss.tchncs.de ) 12•11 months agoMy personal gut feeling: the maintainer will just shoot this attempt down and close the PR.
Happy to be proven wrong here though.
miniu ( @miniu@programming.dev ) 2•11 months agoAuthor of the PR made it opt-out. I think it may be enough of the compromise for the maintainer.
TehPers ( @TehPers@beehaw.org ) 1•11 months agoHas dtolnay shown a strong sentiment towards having it be on always? I can’t imagine there’s much reason not to make it configurable.
d_k_bo ( @d_k_bo@feddit.de ) 3•11 months agoThe precompiled implementation is the only supported way to use the macros that are published in serde_derive. If there is implementation work needed in some build tools to accommodate it, someone should feel free to do that work
~ dtolnay (source)
That sounds like he doesn’t like to (re-)introduce any other options.
TehPers ( @TehPers@beehaw.org ) 3•11 months agoThat seems a bit concerning. At first I was under the impression the binary was compiled on the user’s machine, but once I saw that it’s distributed with serde I can see why people are upset.
d_k_bo ( @d_k_bo@feddit.de ) 3•11 months agoI was under the impression the binary was compiled on the user’s machine
That’s how procedural macros used to and were intended to work.
Buttons ( @Buttons@programming.dev ) English3•11 months agoThe precompiled implementation is the only supported way to use the macros that are published in serde_derive
That statement is straight up gaslighting.
The precompiled binary is only provided for one platform, Linux. Windows does not use a precompiled binary but compiles its own from the source. How can he claim it’s the “only supported way”, when for most platforms he is doing it another way? Also, the crate, throughout most of its life, has been doing it another way.
- argv_minus_one ( @argv_minus_one@beehaw.org ) English9•11 months ago
So, the general unsuspecting public will still be executing this potentially-malicious binary, and only those in the know will be able to avoid it?
anlumo ( @anlumo@feddit.de ) English6•11 months agoOnly if that PR gets accepted, which is unlikely.
anlumo ( @anlumo@feddit.de ) English1•11 months agoYeah, that was very unexpected.