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

Adds support for Time.zone via ActiveSupport::TimeWithZone #34

Closed
wants to merge 1 commit into from
Closed

Adds support for Time.zone via ActiveSupport::TimeWithZone #34

wants to merge 1 commit into from

Conversation

keichan34
Copy link

Because EXIF does not include timezone information, the user should be able to set what timezone the times are parsed in. ActiveSupport has a nice way to do this:

Time.use_zone('UTC') { exif = EXIFR::JPEG.new 'image.jpeg' }

@keichan34
Copy link
Author

Ah, I just saw your comment on not wanting to introduce a dependency on ActiveSupport. Sorry about that!

@keichan34 keichan34 closed this Sep 29, 2014
@remvee
Copy link
Owner

remvee commented Sep 30, 2014

See bbd6e89 for a solution.

@keichan34
Copy link
Author

Ah, awesome. Thanks!

ombr pushed a commit to ombr/exifr that referenced this pull request Jun 7, 2021
fix: Firefox > 77 not checked for auto rotation
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