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

Crash caused by taking a screenshot #2603

Closed
SmallJoker opened this issue Apr 4, 2015 · 5 comments

Comments

Projects
None yet
3 participants
@SmallJoker
Copy link
Member

commented Apr 4, 2015

Available errors: None; crashes without any log mesage or .dmp file.

Possible source of this bug:
862d4ea : screenshot_YYYY-MM-DDTHH::MM::SS[-serial].png
: is not allowed in filenames.

@nerzhul

This comment has been minimized.

Copy link
Member

commented Apr 4, 2015

":" is not allowed on windows if i remember. We must be compliant with both Linux/UNIX, Windows and Mac OSX

@nerzhul

This comment has been minimized.

Copy link
Member

commented Apr 4, 2015

@Zeno- because you are the owner of the screenshot patch, could you fix the path to be compliant with all OS ?

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented Apr 6, 2015

High priority? Ok, onto it!

@Zeno- Zeno- closed this in 43aab61 Apr 6, 2015

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented Apr 6, 2015

@Zeno- Zeno- reopened this Apr 6, 2015

@Zeno-

This comment has been minimized.

Copy link
Contributor

commented Apr 6, 2015

oops ... didn't mean to re-open

@Zeno- Zeno- closed this Apr 6, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.