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

Do we still need the BROWSER var? #44

Closed
philmmanjaro opened this issue Apr 28, 2016 · 9 comments
Closed

Do we still need the BROWSER var? #44

philmmanjaro opened this issue Apr 28, 2016 · 9 comments
Milestone

Comments

@philmmanjaro
Copy link
Member

Sent by email: 27.04.2016 at 23:20 - from Dax Gahr:
I've recently been having issues with too many resources being used up for
apparently no reason. I figured out that xdg-open is spawning a ton of
processes of itself.

It looks like in /etc/profile, BROWSER is set to "xdg-open", which is
what's causing the recursive call issue. I tested this by typing "xdg-open
http://www.google.com/", and sure enough, a ton of "xdg-open" processes
started appearing. I can fix this myself by setting BROWSER to something
else, but I was wondering if this was set in /etc/profile intentionally??

@oberon-manjaro
Copy link
Member

oberon-manjaro commented Apr 28, 2016

At the moment we also export it from /etc/environment ...
People using WMs will certainly be happy if that can be dropped ;)

@noobpurple
Copy link

I also found it in my ~/.bashrc, by the way. Commenting them out fixed the issue.

@oberon-manjaro
Copy link
Member

@noobpurple , which edition do you use? I assume that in your .bashrc the variable was not set to xdg-open but to a specific browser?

@noobpurple
Copy link

I use manjaro-net-15.12 with i3 installed. And, no, it was originally set to xdg-open in my .bashrc.

@oberon-manjaro
Copy link
Member

OK. Especially for WMs that is certainly not a good idea 😜

@philmmanjaro
Copy link
Member Author

So we drop it then.

@philmmanjaro philmmanjaro added this to the 16.06-rc1 milestone May 2, 2016
@philmmanjaro
Copy link
Member Author

@Oberon2007: well we have it also in thus and in calamares. Cli-Installer might be also the case ...

@philmmanjaro
Copy link
Member Author

Yep, cli has it also.

@oberon-manjaro
Copy link
Member

:bowtie:

philmmanjaro added a commit to manjaro/packages-extra that referenced this issue May 4, 2016
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

No branches or pull requests

3 participants