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 deprecations from chrono 0.4.23. #336

Merged
merged 3 commits into from
Nov 20, 2022
Merged

Fix deprecations from chrono 0.4.23. #336

merged 3 commits into from
Nov 20, 2022

Conversation

metasim
Copy link
Contributor

@metasim metasim commented Nov 14, 2022

  • I agree to follow the project's code of conduct.
  • I added an entry to CHANGES.md if knowledge of this change could be valuable to users.

Fixes #335

@lnicola
Copy link
Member

lnicola commented Nov 15, 2022

Ah, and we don't even support OFTTime, do we?

Looks good, I suppose, but I'd like someone else to take a look, as I'm pretty rusty on both the chrono API and the OGR data types.

.cargo/config.toml Outdated Show resolved Hide resolved
@metasim
Copy link
Contributor Author

metasim commented Nov 15, 2022

@jdroenner, @ttencate or @ChristianBeilschmidt: would one of you have the time to review this? 🙏

extra space

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>
@ttencate
Copy link
Contributor

@metasim Not me, sorry. I'm just an occasional contributor, not a maintainer, and don't have time to do more at the moment :)

@metasim
Copy link
Contributor Author

metasim commented Nov 16, 2022

bors r+

bors bot added a commit that referenced this pull request Nov 16, 2022
336: Fix deprecations from chrono 0.4.23. r=metasim a=metasim

- [x] I agree to follow the project's [code of conduct](https://github.com/georust/gdal/blob/master/CODE_OF_CONDUCT.md).
- [x] I added an entry to `CHANGES.md` if knowledge of this change could be valuable to users.
---

Fixes #335 

Co-authored-by: Simeon H.K. Fitch <fitch@astraea.io>
@bors
Copy link
Contributor

bors bot commented Nov 16, 2022

Build failed:

@metasim
Copy link
Contributor Author

metasim commented Nov 16, 2022

Blocked by #337

@michaelkirk
Copy link
Member

I am so sorry. 🙈

@metasim
Copy link
Contributor Author

metasim commented Nov 16, 2022

I am so sorry. 🙈

LoL 🔥 🧑‍🚒 😆

@metasim metasim added dependencies Pull requests that update a dependency file maintenance labels Nov 19, 2022
metasim added a commit to metasim/gdal that referenced this pull request Nov 19, 2022
@metasim
Copy link
Contributor Author

metasim commented Nov 20, 2022

bors r+

@bors bors bot merged commit 9cbaedb into georust:master Nov 20, 2022
@metasim metasim deleted the fix/335 branch November 20, 2022 18:28
metasim added a commit to metasim/gdal that referenced this pull request Nov 20, 2022
* master:
  Hold back geo-types version; deprecations addressed in georust#339
  Fixes deprecations introduced in geo-types v0.7.8.
  Hold back chrono version, as it's addressed in georust#336
  Update .cargo/config.toml
  Fix deprecations from chrono 0.4.23.
@metasim metasim mentioned this pull request Nov 20, 2022
2 tasks
@metasim metasim mentioned this pull request Dec 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chrono 0.4.23 deprecations
4 participants