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

Add preserve metadata flag #175

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

gtuk
Copy link

@gtuk gtuk commented Apr 4, 2017

No description provided.

@sunatthegilddotcom
Copy link

@gtuk Did you fix the meta data missing issue? Thank you very much for your help!

@gtuk
Copy link
Author

gtuk commented Apr 5, 2017

@sunatthegilddotcom yes there will be an additional argument "--preserve-exif" to preverse the exif metadata

@wsun-clgx
Copy link

wsun-clgx commented Apr 6, 2017

@gtuk Thank you very much for your help and fix this issue in so short of time. , You are the best! I did the git pull from your repo, and compiled the code, and it complaint "Unknown commandline flag: --preserve-exif". Can you please help me? Thanks a lot. Here is what I used

~/gtuk/guetzli/bin/Release$ ./guetzli --preserve-exif x9.jpg x9.jpg.jpg
Unknown commandline flag: --preserve-exif
Guetzli JPEG compressor. Usage:
guetzli [flags] input_filename output_filename

Flags:
--verbose - Print a verbose trace of all attempts to standard output.
--quality Q - Visual quality to aim for, expressed as a JPEG quality value.
Default value is 95.
--memlimit M - Memory limit in MB. Guetzli will fail if unable to stay under
the limit. Default limit is 6000 MB.
--nomemlimit - Do not limit memory usage.

@gtuk
Copy link
Author

gtuk commented Apr 6, 2017

@liankui It seems that you used the master branch of my fork. You have to switch the branch to the "preserve-exif-flag" branch and then compile it again. Now it should work. Let me know if you need any further assistants.

@wsun-clgx
Copy link

@gtuk Thank you very much for your guidance. I was able to pull the right branches and now it works beautifully. Great Job and Thanks a lot. You are great!

@sepatus88
Copy link

sepatus88 commented Apr 7, 2017 via email

@swrobel
Copy link

swrobel commented Jun 28, 2017

Could @szabadka @robryk or someone else from the guetzli team please look at this patch because it's a major issue for photographers wanting to use guetzli on their files.

@AMG39
Copy link

AMG39 commented Jun 28, 2017 via email

@maltris
Copy link

maltris commented May 18, 2019

Any update on this, will it be merged after 2 years, finally?

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