Skip to content

/rust

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

Remove build date from the output of --version #25821

Merged
merged 1 commit into from Jun 2, 2015

Conversation

Reviewers
No reviews
Assignees

@brson brson

Labels
beta-accepted
Projects
None yet
Milestone
No milestone
6 participants
@jooert @rust-highfive @brson @bors @alexcrichton @pcwalton
@jooert
Copy link
Contributor

jooert commented May 27, 2015

Closes #25812.
@jooert
Remove build date from the output of --version 
Closes #25812.
Loading status checks…
c408667

@rust-highfive rust-highfive assigned pcwalton May 27, 2015

@rust-highfive

This comment has been minimized.

Sign in to view
Copy link
Collaborator

rust-highfive commented May 27, 2015

r? @pcwalton

(rust_highfive has picked a reviewer for you, use r? to override)
@jooert

This comment has been minimized.

Sign in to view
Copy link
Contributor Author

jooert commented May 27, 2015

r? @brson

@rust-highfive rust-highfive assigned brson and unassigned pcwalton May 27, 2015

@brson

This comment has been minimized.

Sign in to view
Copy link
Contributor

brson commented Jun 1, 2015

@bors r+ rollup
@bors

This comment has been minimized.

Sign in to view
Copy link
Contributor

bors commented Jun 1, 2015

📌 Commit c408667 has been approved by brson

Manishearth added a commit to Manishearth/rust that referenced this pull request Jun 2, 2015

@Manishearth
Rollup merge of rust-lang#25821 - jooert:remove_build_date, r=brson 
Closes rust-lang#25812.
4f5a744

@Manishearth Manishearth referenced this pull request Jun 2, 2015

Merged

Rollup of 9 pull requests #25958

bors added a commit that referenced this pull request Jun 2, 2015

@bors
Auto merge of #25958 - Manishearth:rollup, r=Manishearth 
- Successful merges: #25751, #25821, #25920, #25932, #25933, #25936, #25941, #25949, #25951
- Failed merges:
Loading status checks…
48e9ef6

@bors bors merged commit c408667 into rust-lang:master Jun 2, 2015
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jooert jooert deleted the jooert:remove_build_date branch Jun 2, 2015

@brson brson added the beta-nominated label Jun 3, 2015

@brson

This comment has been minimized.

Sign in to view
Copy link
Contributor

brson commented Jun 3, 2015

Nominating for beta because this breaks parsing of rustc --version so better to do it quickly.

@alexcrichton alexcrichton added the T-tools label Jun 3, 2015

@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member

alexcrichton commented Jun 4, 2015

cc @rust-lang/tools

I agree this should be backported
@alexcrichton

This comment has been minimized.

Sign in to view
Copy link
Member

alexcrichton commented Jun 10, 2015

Accepting for the backport

@alexcrichton alexcrichton added the beta-accepted label Jun 10, 2015

@alexcrichton alexcrichton referenced this pull request Jun 10, 2015

Merged

Backport accepted PRs to beta #26161

@alexcrichton alexcrichton removed the beta-nominated label Jun 11, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.