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

Don't evaluate trusted variable in *ExportCmd::parse as it will be fa… #20

Closed
wants to merge 1 commit into from

Conversation

@mbroemme
Copy link
Contributor

commented Dec 2, 2015

…lse alway for SetEnv, Export and SetResourceValue and never execute them via fluxbox-remote.

It is required to allow resource variable changes, for example:

$> fluxbox-remote 'SetResourceValue session.screen0.toolbar.placement BottomLeft'
$> fluxbox-remote 'Reconfig'

Without the fix the first command will do nothing as it always will return 0.

…lse alway for SetEnv, Export and SetResourceValue and never execute them via fluxbox-remote.
@mgumz

This comment has been minimized.

Copy link
Member

commented Jan 4, 2016

make the commit-subject shorter, please.

@mgumz

This comment has been minimized.

Copy link
Member

commented May 28, 2016

@mbroemme if you could make the commit-subject to fit into the ~60 columns range i would accept your otherwise fine patch.

@mgumz

This comment has been minimized.

Copy link
Member

commented Mar 11, 2018

rephrased, rebased + merged.

@mgumz mgumz closed this Mar 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.