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

Fix spec for running under 32-bit Linux #416

Merged
merged 1 commit into from
Jun 25, 2020

Conversation

lugia-kun
Copy link
Contributor

  • 1900/01/01 is not a representable date on 32-bit Linux and restricted to the oldest date which is representable.

See crystal-lang/crystal#9542 for more info.

* Restricted to the valid time range for that system
@@ -419,7 +419,7 @@ describe "install" do

with_shard(metadata) do
run "shards install"
File.touch "shard.lock", Time.utc(1900, 1, 1)
File.touch "shard.lock", Time.utc(1901, 12, 14)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why such an old time is needed here 🤔

Can't we just use a date from last year or something like that?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Something pre 1970 makes sense I guess in case somebody boots their system without a hardware clock and it starts from 0 :D

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good luck with doing whatever on such machine (think SSL certificates with specific timeframe) ;)

@bcardiff bcardiff merged commit 6281f44 into crystal-lang:master Jun 25, 2020
@waj waj added this to the v0.12.0 milestone Jul 28, 2020
taylor pushed a commit to vulk/shards that referenced this pull request Aug 11, 2020
* Restricted to the valid time range for that system
f-fr pushed a commit to f-fr/shards that referenced this pull request Jan 2, 2021
* Restricted to the valid time range for that system
@lugia-kun lugia-kun deleted the fix/spec-on-32bit-linux branch April 15, 2021 14:57
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.

None yet

7 participants