Comments (6)
Hm, the option handling should likely be ported to argparse.ArgumentParser to replace this manual stuff. According to the code, any arguments that are not --list
, --help
or --version
result in the argument(s) getting interpreted as desktop environment names, the choices of which is supposed to be only one or many of GNOME, KDE, XFCE, ROX and Old. So the correct fix would likely be to enforce this constraint. It seems the current code just lets it slide, resulting in this weird error because something indeterminable is getting executed. This is an interface bug.
I wonder if this tool is still required in its current form? Perhaps we can replace it with a tool from some Debian package? Surely there must be a way to launch all xdg autostart stuff apart from custom logic to this extent?
Next, is it normal for this tool to be executed like you did – manuell, in the terminal or is it usually hidden from users?
Had I known there was still a py2 program in a place like this, I'd rewritten it long ago. Perhaps it's a little late for that now :)
from bunsen-configs.
The autostart file has this instruction in the header comments:
## Run 'bl-xdg-autostart --list' to list any XDG autostarted programs.
This is how the bug came to light when a BL user used -h instead of --help.
bl-xdg-autostart is basically a re-named openbox-xdg-autostart
from bunsen-configs.
Next, is it normal for this tool to be executed like you did – manuell, in the terminal or is it usually hidden from users?
It's run automatically on every startup, but users have the --list option to check what apps are being xdg autostarted and why/why not. That's quite a useful function I use myself from time to time.
Had I known there was still a py2 program in a place like this, I'd rewritten it long ago. Perhaps it's a little late for that now.
There are still some (many?) py2 scripts among our utilities, and upgrading them to py3 has long been on my todo list. It's not something I can do myself, though, being a Python beginner. Would you like me to send you a list in case any of them can be easily fixed? It's not too late, as long as no outside interfaces change.
from bunsen-configs.
if len(argv) == 1 or argv[1] not in arg_list:
I think this needs changing. bl-xdg-autostart (like its openbox parent) is usually launched without any arguments in the user startup. With the condition len(argv) == 1
then it just returns failure and displays help instead of doing the autostarting.
EDIT:
Tentative suggestion:
if len(argv) == 2 and argv[1] not in arg_list:
but bear in mind 2ion's proposal for more extensive rewrite.
from bunsen-configs.
OK I didn't think of the autostarted bl-xdg-autostart :(
But if the user runs it with no script args it fails with errors, so that situation needs error-trapping as well.
from bunsen-configs.
if the user runs it with no script args it fails with errors
I think that's because of some of the previously autostarted processes objecting to more than one instance being started up. (Others will happily start a second instance.)
There might occasionally be situations - eg trouble-shooting - when a user actually wants to run the xdg autostart manually, no? Anyway, how to distinguish between that and the startup script? Test for a controlling terminal? We might just have to let this one be, though it's messy of course.
from bunsen-configs.
Related Issues (20)
- Super+w doesn't work on BL-Lithium ARM install... HOT 1
- Hyperlinks (e.g. in PDFs) open in non-default browser HOT 10
- skel/.config/bunsen/autostart: replace round brackets with curly brackets? HOT 4
- i want to ask HOT 2
- Change directory HOT 1
- Tint2 gitlab has new adress HOT 2
- add menu for hw-probe HOT 2
- Jgmenu systemreport shows too little information HOT 2
- [Suggestion] Thunar keybind / F4 to Open terminal here HOT 3
- prepend.csv: Illegal quoting in line 6 HOT 3
- installing .deb programs bugging my mousepad HOT 3
- ALT key is (maybe) being eaten by the desktop HOT 4
- bl-user-setup doesn't work with domain join users HOT 4
- skel/.config/blob/: tint2 config file names do not match those in skel/.config/tint2 HOT 1
- lightdm: Set greeter-hide-users=false to show available users in greeter HOT 5
- Update bl-mpv for mpv=0.23 and provide skel/.config/mpv/mpv.conf HOT 2
- mpv config: window is forced even when running mpv in terminal HOT 6
- mpv config: BL uses the deprecated [pseudo-gui] profile HOT 2
- /grub2/i386-pc/normal.mod not Found when Booting HELIUM5 AMD64 Installation from an USB-drive HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bunsen-configs.