Skip to content

Fix Errno issue with Crystal 0.34.0 and release v0.20.0#499

Merged
paulcsmith merged 3 commits into
masterfrom
pcs/0.34.0-support
Apr 9, 2020
Merged

Fix Errno issue with Crystal 0.34.0 and release v0.20.0#499
paulcsmith merged 3 commits into
masterfrom
pcs/0.34.0-support

Conversation

@paulcsmith
Copy link
Copy Markdown
Member

Errno was renamed in 0.34.x crystal-lang/crystal#8885

Instead we should just ignore non existent file. That way it works in
0.33 and 0.34

@paulcsmith
Copy link
Copy Markdown
Member Author

Issue with shards not resolving since we can't override the dependency. I don't want to go create separate branches for every dependency so they point to lucky master. So hoping crystal-lang/shards#299 gets added.

If not added by 0.34.0 (which seems quite likely it won't be since 0.34 is pretty close) then I'll wait to get a version of lucky locked down for 0.34.0, then update this to point to that branch and should hopefully be able to test it out

Errno was renamed in 0.34.x crystal-lang/crystal#8885

Instead we should just ignore non existent file. That way it works in
0.33 and 0.34
@paulcsmith paulcsmith force-pushed the pcs/0.34.0-support branch from b8df0ec to cb94d3c Compare April 7, 2020 02:41
@paulcsmith paulcsmith changed the title Fix Errno issue with Crystal 0.34.0 Fix Errno issue with Crystal 0.34.0 and release v0.20.0 Apr 8, 2020
@paulcsmith paulcsmith force-pushed the pcs/0.34.0-support branch from b5e361d to 9a19526 Compare April 8, 2020 22:09
@paulcsmith paulcsmith force-pushed the pcs/0.34.0-support branch from 9a19526 to 112179e Compare April 8, 2020 22:21
@paulcsmith paulcsmith merged commit fea14b4 into master Apr 9, 2020
@paulcsmith paulcsmith deleted the pcs/0.34.0-support branch April 9, 2020 00:05
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.

2 participants