@Kr1ss That was impressive! Nice patch!
Search Criteria
Package Details: ventoy-bin 1.1.05-1
Package Actions
| Git Clone URL: | https://aur.archlinux.org/ventoy-bin.git (read-only, click to copy) |
|---|---|
| Package Base: | ventoy-bin |
| Description: | A new bootable USB solution |
| Upstream URL: | http://www.ventoy.net |
| Keywords: | boot image iso multiboot usb |
| Licenses: | GPL-3.0-or-later |
| Conflicts: | ventoy |
| Provides: | ventoy |
| Submitter: | DuckSoft |
| Maintainer: | DuckSoft (KokaKiwi, yochananmarqos, Kr1ss) |
| Last Packager: | yochananmarqos |
| Votes: | 383 |
| Popularity: | 5.25 |
| First Submitted: | 2020-05-19 06:14 (UTC) |
| Last Updated: | 2025-02-24 18:30 (UTC) |
Dependencies (7)
- bash (bash-devel-gitAUR, bash-gitAUR)
- dosfstools (dosfstools-gitAUR)
- util-linux (util-linux-aesAUR, util-linux-selinuxAUR)
- xz (xz-gitAUR)
- gtk3 (gtk3-no_deadkeys_underlineAUR, gtk3-classicAUR, gtk3-classic-xfceAUR, gtk3-patched-filechooser-icon-viewAUR) (optional) – GTK3 GUI
- polkit (polkit-gitAUR, polkit-consolekitAUR) (optional) – run GUI from application menu
- qt5-base (qt5-base-gitAUR, qt5-base-headlessAUR) (optional) – Qt5 GUI
Required by (0)
Sources (10)
DuckSoft commented on 2021-02-28 17:12 (UTC)
Kr1ss commented on 2021-02-28 16:57 (UTC)
So I've extended the patch so that both the ventoy CLI and webUI are now logging to /var/log instead of /opt/ventoy. There's also now /usr/bin/ventoyweb
to start the webserver.
Unfortunately we do still need root in order to write to the USB storage. I guess we can now remove the post_install note though, since permanent filesystem trees aren't touched any more.
Kr1ss commented on 2021-02-27 21:16 (UTC)
Ooops. Thanks ! :)
yochananmarqos commented on 2021-02-27 21:15 (UTC)
@Kr1ss: You forgot something. ;)
Kr1ss commented on 2021-02-27 19:25 (UTC)
Okay, I'll push a pkgrel bump and include an install file with a note.
DuckSoft commented on 2021-02-27 18:37 (UTC) (edited on 2021-02-27 18:39 (UTC) by DuckSoft)
No offense but I think Ventoy author is not aware of that FHS, XDG or anything else. I will write a report to the upstream and hopefully this will improve.
Currently let's just include that WebUI directory and leave everything as-is, @Kriss. Leave a caveats note in install.sh postinstall may also be helpful.
ps: My keyboard is broken and I typed these on my phone...
Kr1ss commented on 2021-02-27 18:27 (UTC) (edited on 2021-02-27 18:31 (UTC) by Kr1ss)
@DuckSoft @yochananmarqos Any idea why this would need root anyways, besides of writing to /opt tree ?
i may want to split the WebUI outside ventoy-bin, then patch the source & build that webui as another package.
That sounds reasonable.
DuckSoft commented on 2021-02-27 18:20 (UTC) (edited on 2021-02-27 18:22 (UTC) by DuckSoft)
@yochananmarqos inspected the code carefully... a quick and dirty fix is to put WebUI directory back then.
But there's something awful: it's executed as root, but he's putting crap log.txt directly in the binary directory, which is...
I'd try hard to fix the package, but if things are going worse, i may want to split the WebUI outside ventoy-bin, then patch the source & build that webui as another package.
yochananmarqos commented on 2021-02-27 18:09 (UTC)
Does the new Ventoy Web GUI work for anyone else?
https://www.ventoy.net/en/doc_linux_webui.html
sudo /opt/ventoy/VentoyWeb.sh
I just get a 404 error opening http://127.0.0.1:24680
Pinned Comments