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

winusb: Use pkexec instead of gksudo for privilege escalation #50648

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bugcrazy
Copy link
Contributor

@bugcrazy bugcrazy commented Jun 2, 2024

Testing the changes

  • I tested the changes in this PR: YES

@bugcrazy
Copy link
Contributor Author

bugcrazy commented Jun 2, 2024

Patch for frontend-wxgtk: WoeUSB/WoeUSB-frontend-wxgtk@94f1b08

@classabbyamp
Copy link
Member

that commit has been included in releases, why not just update the package?

@bugcrazy
Copy link
Contributor Author

bugcrazy commented Jun 2, 2024

that commit has been included in releases, why not just update the package?

There have been no more winusb releases since 2015, the commit is more recent, derived from the code of another with the same code base.

@classabbyamp
Copy link
Member

then perhaps winusb should be removed

@bugcrazy
Copy link
Contributor Author

bugcrazy commented Jun 3, 2024

then perhaps winusb should be removed

It can be removed winusb, woeusb is in the repository, it is a fork of winusb, but it is not maintained, woe-cli is a fork of woeusb, there is a more updated project is woeusb-ng, it is not in the repository.

This explains the status of the projects:
https://github.com/slacka/WoeUSB?tab=readme-ov-file

@sgn
Copy link
Member

sgn commented Jun 5, 2024

Let's remove winusb, and make woeusb provides winusb. Maybe winusb as symlink to woeuse?

@bugcrazy
Copy link
Contributor Author

bugcrazy commented Jun 6, 2024

Let's remove winusb, and make woeusb provides winusb. Maybe winusb as symlink to woeuse?

There could be a symlink, the command between the two, works the same, just changes the name.

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

3 participants