Skip to content

Add the build date to the reported version. #21957 #22201

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Feb 16, 2015

Conversation

brson
Copy link
Contributor

@brson brson commented Feb 12, 2015

rustc --version says

rustc 1.0.0-dev (d0e82a68a 2015-02-05 14:38:56 -0800) (built 2015-02-11)

@brson
Copy link
Contributor Author

brson commented Feb 12, 2015

cc rust-lang/cargo#1292

@rust-highfive
Copy link
Contributor

r? @nick29581

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

@brson
Copy link
Contributor Author

brson commented Feb 12, 2015

This is my attempt to solve the confusion around the date reported by --version. Open to other ideas. Needs to be merged with the corresponding Cargo PR.

@nrc
Copy link
Member

nrc commented Feb 12, 2015

lgtm, you can take that as r+ if you like or wait to see if anyone has a better idea...

@brson
Copy link
Contributor Author

brson commented Feb 12, 2015

I'd like to get rid of the useless '14:38:56 -0800' part of the commit date but git doesn't seem to support printing just the date.

@iKevinY
Copy link
Contributor

iKevinY commented Feb 12, 2015

It seems like you can print just the date using git log -1 --date=short --pretty=format:'%cd'.

@brson
Copy link
Contributor Author

brson commented Feb 13, 2015

@iKevinY Thanks for the tip!

@brson
Copy link
Contributor Author

brson commented Feb 13, 2015

I've updated the PR to not put the time and tz offset in the version.

rustc --version says

```
rustc 1.0.0-dev (d0e82a6 2015-02-05) (built 2015-02-11)
```
@brson
Copy link
Contributor Author

brson commented Feb 13, 2015

@bors: r=nick29581 rollup

@bors
Copy link
Collaborator

bors commented Feb 13, 2015

🙀 You have the wrong number! Please try again with effad62.

@brson
Copy link
Contributor Author

brson commented Feb 13, 2015

@bors: r=nick29581 effad62 rollup

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Feb 14, 2015
rustc --version says

```
rustc 1.0.0-dev (d0e82a6 2015-02-05 14:38:56 -0800) (built 2015-02-11)
```
steveklabnik added a commit to steveklabnik/rust that referenced this pull request Feb 15, 2015
rustc --version says

```
rustc 1.0.0-dev (d0e82a6 2015-02-05 14:38:56 -0800) (built 2015-02-11)
```
Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 15, 2015
rustc --version says

```
rustc 1.0.0-dev (d0e82a6 2015-02-05 14:38:56 -0800) (built 2015-02-11)
```
@bors bors merged commit effad62 into rust-lang:master Feb 16, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants