search

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

Screenshot not being saved when there is no Desktop folder #54

Closed burakcoskun closed 5 years ago

burakcoskun commented 5 years ago

To reproduce mv Desktop to somewhere else and try to save screenshot with deepin with default options. Some linux distros like arch doesn't come with a Desktop folder.

BLumia commented 5 years ago

What's the result if you run xdg-user-dir DESKTOP from terminal?

If the result is a path which is not existed, try run xdg-user-dirs-update or manually create the path?

burakcoskun commented 5 years ago

Currently I don't have access to my machine, but if I manually create Desktop folder it works.

Not a big issue but shouldn't tool create the path if it doesn't exist?

BLumia commented 5 years ago

Not a big issue but shouldn't tool create the path if it doesn't exist?

Yep, I did a PR(#55) about this, but I want to make sure your xdg-user-dir configuration is correct :)

burakcoskun commented 5 years ago

Perfect. Sure then I will check my xdg-user-dir :).

Thanks