this post was submitted on 19 Aug 2023
31 points (94.3% liked)

Rust

6038 readers
16 users here now

Welcome to the Rust community! This is a place to discuss about the Rust programming language.

Wormhole

[email protected]

Credits

  • The icon is a modified version of the official rust logo (changing the colors to a gradient and black background)

founded 1 year ago
MODERATORS
top 15 comments
sorted by: hot top controversial new old
[–] [email protected] 13 points 1 year ago (3 children)

My personal gut feeling: the maintainer will just shoot this attempt down and close the PR.

Happy to be proven wrong here though.

[–] [email protected] 8 points 1 year ago (2 children)

dtolnay isn't the only maintainer though. oli-obk is also a maintainer and is participating in the discussions on the PR.

[–] [email protected] 6 points 1 year ago* (last edited 1 year ago) (1 children)

I don't think oli-obk has any say in this as they stated the following:

Because this crate is owned by dtolnay and he may do with it as he wishes. My personal opinions on the topic are irrelevant.

Source

[–] [email protected] 3 points 1 year ago

You're right, I missed that. That's unfortunate.

[–] [email protected] 3 points 1 year ago

Technically true, but he has two merge commits in last three years. Not sure that will still give him much of a pull.

[–] miniu 3 points 1 year ago

Author of the PR made it opt-out. I think it may be enough of the compromise for the maintainer.

[–] [email protected] 1 points 1 year ago (1 children)

Has dtolnay shown a strong sentiment towards having it be on always? I can't imagine there's much reason not to make it configurable.

[–] [email protected] 3 points 1 year ago (2 children)

The 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.

[–] [email protected] 3 points 1 year ago (1 children)

That 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.

[–] [email protected] 3 points 1 year ago

I 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 3 points 1 year ago* (last edited 1 year ago)

The 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.

[–] [email protected] 8 points 1 year ago (1 children)

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?

[–] [email protected] 6 points 1 year ago (1 children)

Only if that PR gets accepted, which is unlikely.

[–] [email protected] 2 points 1 year ago (1 children)
[–] [email protected] 0 points 1 year ago

Yeah, that was very unexpected.