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

I added data type(json) for client communicate easily with qtile server. #868

Closed
wants to merge 2 commits into from

Conversation

sux2mfgj
Copy link
Contributor

I read comment in libqtile/ipc.py. I agree that qtile don't permit arbitrary code execution. but I want to access qtile easily from 3rd party client. and I think these client make qtile easily of implementation checking at server.

I'm sorry if my English sentences are incorrect.

options, args = parser.parse_args()

client = command.Client(options.socket)
print("is_json: " + str(options.is_json))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove this line.

@flacjacket
Copy link
Member

I fixed up some merge conflicts, squashed, and merged this, thanks!

@sux2mfgj
Copy link
Contributor Author

I'm sorry for I didn't send better code, and thank you!

@sux2mfgj sux2mfgj deleted the communicate_json branch February 29, 2016 20:26
@flacjacket
Copy link
Member

No problem, there were just a few changes in the master branch that made this not apply cleanly (you made changes to bin/qsh, but that was moved to libqtile/scripts/qsh.py), all I did was transfer your changes over to the new location.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants