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

Use IO.getModifiedTimeOrZero(file) calls #470

Merged
merged 3 commits into from
Dec 22, 2017
Merged

Conversation

cunei
Copy link

@cunei cunei commented Dec 18, 2017

There are just too many instances in which sbt's code relies on
the lastModified/setLastModified semantics, so instead of moving
to get/setModifiedTime, we use new IO calls that offer the new
timestamp precision, but retain the old semantics.

@cunei cunei requested review from eed3si9n and dwijnand December 18, 2017 16:30
Antonio Cunei added 2 commits December 22, 2017 01:36
There are just too many instances in which sbt's code relies on
the `lastModified`/`setLastModified` semantics, so instead of moving
to `get`/`setModifiedTime`, we use new IO calls that offer the new
timestamp precision, but retain the old semantics.
@typesafe-tools
Copy link

The validator has checked the following projects against Scala 2.12,
tested using dbuild, projects built on top of each other.

Project Reference Commit
sbt pull/3822/head sbt/sbt@94e36a1
zinc pull/470/head 67d3878
io pull/106/head sbt/io@77d3ba2
librarymanagement pull/192/head sbt/librarymanagement@72dd006
util pull/136/head sbt/util@0a1bd5a
website 1.x

✅ The result is: SUCCESS
(restart)

@eed3si9n eed3si9n changed the title Revert *ModifiedTime() calls to *lastModified*() calls Use IO.getModifiedTimeOrZero(file) calls Dec 22, 2017
@eed3si9n
Copy link
Member

I'm going to merge this since it's a module bump.

@eed3si9n eed3si9n merged commit e6b5420 into sbt:1.1.x Dec 22, 2017
@cunei cunei deleted the wip-milliBis branch December 23, 2017 02:26
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.

3 participants