martyr-deepin / deepin-screenshot

Deprecated (migrated into deepin-screen-recorder after v20), consider create and maintain your own fork if you love it.
https://www.deepin.org/original/deepin-screenshot/
GNU General Public License v3.0
346 stars 102 forks source link

find a bug in v4.0.14 and v4.0.15: --save-path command line option is broken #30

Closed ybyygu closed 6 years ago

ybyygu commented 6 years ago

The cmdline option to set output file path is broken:

deepin-screenshot -s /tmp/a.png ls /tmp/a.png ls: cannot access '/tmp/a.png': No such file or directory

It works if not specifiying the saving path.

The last good version is 4.0.11.

System: Arch Linux, 4.16.13-1-zen

ybyygu commented 6 years ago

looks like it already got fixed:

https://github.com/linuxdeepin/deepin-screenshot/commit/85b51d63cc845dd67e8bf8c77a8cdf1b6d2b75ce

waiting for the new release

ybyygu commented 6 years ago

not fixed actually..., the bug is still there :(

What I tried:

  1. update to v4.0.15 using Archlinux package: deepin-screenshot-4.0.15-1-x86_64.pkg.tar.xz
  2. built the binary from git source codes.

None of them works as expected.

> ./deepin-screenshot -s /tmp/a.png 
> ll /tmp/a.png
ls: cannot access '/tmp/a.png': No such file or directory

git status: [ybyygu@ARCHTower] ~/Incoming/scratch/deepin-screenshot 16:49:53

git status On branch master Your branch is up to date with 'origin/master'.

ybyygu commented 6 years ago

fixed in v4.0.16