Issue 371 in webp: gif2webp doesn't take stdin as argument.

[rhastant… via monorail]

Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 371 by rhastant...@instartlogic.com: gif2webp doesn't take stdin as argument.
https://bugs.chromium.org/p/webp/issues/detail?id=371

What steps will reproduce the problem?
1. cat 109.gif | gif2webp -lossy -q 100 -m 4 -o - -- - > 109-opt.webp


What is the expected output? What do you see instead?
Expected to see a webp file.
Get an error:
GIFLib Error 101: Failed to open given file
cat: write error: Broken pipe


What version of the product are you using? On what operating system?
gif2webp 0.6.1
16.04 ubuntu xenial


Please provide any additional information below.



--
You received this message because:
1. The project was configured to send all issue notifications to this address

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

[bugdro… via monorail]

Comment #1 on issue 371 by bugd...@chromium.org: gif2webp doesn't take stdin as argument.
https://bugs.chromium.org/p/webp/issues/detail?id=371#c1

The following revision refers to this bug:
https://chromium.googlesource.com/webm/libwebp/+/cbde5728c84bf2d5c2fc6fa0f609e46bcf93df1b

commit cbde5728c84bf2d5c2fc6fa0f609e46bcf93df1b
Author: James Zern <jz...@google.com>
Date: Fri Jan 26 11:13:44 2018

gif2webp: add support for reading from stdin

output to stdout is already supported; this matches [cd]webp

BUG=webp:371

Change-Id: Ib1ce1661b16ea792943bca2980f779584e90cc86

[modify] https://crrev.com/cbde5728c84bf2d5c2fc6fa0f609e46bcf93df1b/examples/gif2webp.c
[modify] https://crrev.com/cbde5728c84bf2d5c2fc6fa0f609e46bcf93df1b/man/gif2webp.1

[rhastant… via monorail]

Comment #2 on issue 371 by rhastant...@instartlogic.com: gif2webp doesn't take stdin as argument.
https://bugs.chromium.org/p/webp/issues/detail?id=371#c2

Thanks for the patch, is there a release planned with this patch ?

[rhastant… via monorail]

Comment #3 on issue 371 by rhastant...@instartlogic.com: gif2webp doesn't take stdin as argument.
https://bugs.chromium.org/p/webp/issues/detail?id=371#c3

I verified the patch and it works for animated gifs as well.

[jz… via monorail]

Updates:
Labels: v0.6.2
Status: Fixed

Comment #4 on issue 371 by jz...@google.com: gif2webp doesn't take stdin as argument.
https://bugs.chromium.org/p/webp/issues/detail?id=371#c4

Thanks for the report and verifying the fix. This will make it into the next release, but we don't quite have a target for that just yet. Sometime within the next few months is likely.