Skip to content
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

Ignore line ending on older git versions #57858

Merged
merged 1 commit into from
Jan 24, 2019

Conversation

pietroalbini
Copy link
Member

On Ubuntu 16.04 git 2.7.4 tries to fix the line ending of .png and .ico files, and obviously it ruins them. This PR adds an attribute to those files to ignore which line ending they use.

r? @GuillaumeGomez

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 23, 2019
.gitattributes Outdated Show resolved Hide resolved
On Ubuntu 16.04 git 2.7.4 tries to fix the line ending of .png and .ico
files, and obviously it ruins them. This commit adds an attribute to
those files to properly mark them as binary.
@GuillaumeGomez
Copy link
Member

Thanks!

@bors: r+ rollup

@bors
Copy link
Contributor

bors commented Jan 23, 2019

📌 Commit 645b7c2 has been approved by GuillaumeGomez

@bors
Copy link
Contributor

bors commented Jan 23, 2019

🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 23, 2019
@jethrogb jethrogb mentioned this pull request Jan 23, 2019
Centril added a commit to Centril/rust that referenced this pull request Jan 23, 2019
…uillaumeGomez

Ignore line ending on older git versions

On Ubuntu 16.04 git 2.7.4 tries to fix the line ending of `.png` and `.ico` files, and obviously it ruins them. This PR adds an attribute to those files to ignore which line ending they use.

r? @GuillaumeGomez
bors added a commit that referenced this pull request Jan 24, 2019
Rollup of 11 pull requests

Successful merges:

 - #57179 (Update std/lib.rs docs to reflect Rust 2018 usage)
 - #57730 (Merge visitors in AST validation)
 - #57779 (Recover from parse errors in literal struct fields and incorrect float literals)
 - #57793 (Explain type mismatch cause pointing to return type when it is `impl Trait`)
 - #57795 (Use structured suggestion in stead of notes)
 - #57817 (Add error for trailing angle brackets.)
 - #57834 (Stabilize Any::get_type_id and rename to type_id)
 - #57836 (Fix some cross crate existential type ICEs)
 - #57840 (Fix issue 57762)
 - #57844 (use port 80 for retrieving GPG key)
 - #57858 (Ignore line ending on older git versions)

Failed merges:

r? @ghost
@bors bors merged commit 645b7c2 into rust-lang:master Jan 24, 2019
@pietroalbini pietroalbini deleted the ignore-eol-images branch January 24, 2019 13:02
jethrogb added a commit to jethrogb/rustc-guide that referenced this pull request Jan 14, 2020
jethrogb added a commit to jethrogb/miri that referenced this pull request Jan 14, 2020
jethrogb added a commit to jethrogb/book that referenced this pull request Jan 14, 2020
mark-i-m pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Jan 14, 2020
bors added a commit to rust-lang/miri that referenced this pull request Jan 14, 2020
bors bot added a commit to rust-embedded/book that referenced this pull request Jan 14, 2020
221: Update .gitattributes r=therealprof a=jethrogb

See rust-lang/rust#57858

Co-authored-by: jethrogb <github@jbeekman.nl>
jethrogb pushed a commit to jethrogb/rust-analyzer that referenced this pull request Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants