Skip to content

Use updated local declaration syntax for xfail'd test #9982

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

Use updated local declaration syntax for xfail'd test #9982

wants to merge 2 commits into from

Conversation

danluu
Copy link
Contributor

@danluu danluu commented Oct 21, 2013

This patch is trivial, but it comes with a question. Does Mozilla need to own the copyright on code submitted to Rust?

The reason I ask is that, since the last time I submitted anything to a Mozilla project, I started working at Google, and they (by default) own the copyright on code that I write (even in my spare time). There's a process to assign copyright to another entity, and it should be a formality for something like this, but I'd still have to go through it if that's a requirement for Rust.

Anyway, I'm submitting this incredibly trivial thing because, if I have to go through that process for the first time, I'd like it to be for something that's trivial, so I can see how much of a hassle it is (if any) without having invested much time up front.

I didn't see anything about copyright in the Mozilla contributor's agreement, but I could have easily missed something somewhere else.

@thestinger
Copy link
Contributor

@danluu: Mozilla doesn't require copyright assignment for contributors, but you do need the ability to license the code under the dual license Rust uses.

@danluu
Copy link
Contributor Author

danluu commented Oct 21, 2013

Yes, that's fine. AFAIK, there aren't any limitations on the license, once you get approval to submit a patch (which I did). Thanks for the info!

bors added a commit that referenced this pull request Oct 21, 2013
This patch is trivial, but it comes with a question. Does Mozilla need to own the copyright on code submitted to Rust?

The reason I ask is that, since the last time I submitted anything to a Mozilla project, I started working at Google, and they (by default) own the copyright on code that I write (even in my spare time). There's a process to assign copyright to another entity, and it should be a formality for something like this, but I'd still have to go through it if that's a requirement for Rust.

Anyway, I'm submitting this incredibly trivial thing because, if I have to go through that process for the first time, I'd like it to be for something that's trivial, so I can see how much of a hassle it is (if any) without having invested much time up front.

I didn't see anything about copyright in the Mozilla contributor's agreement, but I could have easily missed something somewhere else.
@thestinger
Copy link
Contributor

@danluu: Ah, this ran into one of the obscure Windows test suite errors. It will work if you mark the main function in the test pub.

@brson
Copy link
Contributor

brson commented Oct 21, 2013

@danluu Thanks! 🌷

bors added a commit that referenced this pull request Oct 22, 2013
This patch is trivial, but it comes with a question. Does Mozilla need to own the copyright on code submitted to Rust?

The reason I ask is that, since the last time I submitted anything to a Mozilla project, I started working at Google, and they (by default) own the copyright on code that I write (even in my spare time). There's a process to assign copyright to another entity, and it should be a formality for something like this, but I'd still have to go through it if that's a requirement for Rust.

Anyway, I'm submitting this incredibly trivial thing because, if I have to go through that process for the first time, I'd like it to be for something that's trivial, so I can see how much of a hassle it is (if any) without having invested much time up front.

I didn't see anything about copyright in the Mozilla contributor's agreement, but I could have easily missed something somewhere else.
@bors bors closed this Oct 22, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 17, 2022
Don't lint `from_over_into` for opaque types

fixes rust-lang#9935

This is stalled until the next sync. The impl in question can't be written on the pinned nightly.

changelog: Don't lint `from_over_into` for opaque types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants