Subject: bug#25759: closed ()

From: [email-masked] (GNU bug Tracking System)
Date: Wed, 08 Nov 2017 21:04:02 +0000

Your bug report

#25759: Should 'surf' depend on 'xprop' and 'dmenu'?

which was filed against the guix package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
25759: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=25759
GNU Bug Tracking System
Contact [email-masked] with problems

From: Mekeor Melire <[email-masked]>
To: [email-masked]
Date: Wed, 08 Nov 2017 22:03:36 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Subject: Should 'surf' depend on 'xprop' and 'dmenu'?
From: Mekeor Melire <[email-masked]>
To: [email-masked]
Date: Thu, 16 Feb 2017 18:12:05 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

The Guix package for the 'surf' browser (from suckless project) does not depend on the packages 'xprop' and 'dmenu'.

Thus, when you only install 'surf' (and do not have 'xprop' and 'dmenu' installed), you can only open a new URL, by re-running 'surf' with that new URL as command-line argument, like "surf example.org". But it's not possible to open a new URL from within 'surf' itself because that requires both 'xprop' and 'dmenu'. This is comprehensible in these lines of the code:
	http://git.suckless.org/surf/tree/config.def.h#n38

On #guix IRC channel, we agreed that 'surf' should depend on these packages:

	<mekeor> the package for the 'surf' browser (from suckless) has a similar issue: if you only install surf, you can't open a new URL within the browser because that requires the xprop package.  what do you think about that case?
	<Sleep_Walker> you mean dmenu, right?
	<mekeor> Sleep_Walker: yeah, that would be awesome
	<Sleep_Walker> ah
	<Sleep_Walker> right
	<mekeor> Sleep_Walker: uhm... i'm not sure. maybe it's both?
	<Sleep_Walker> ctrl+l
	<Sleep_Walker> that IIRC runs dmenu to have URL entered
	<mekeor> Sleep_Walker: it's Ctrl-g for me
	<Sleep_Walker> and yes, that should be fixed as well
	<Sleep_Walker> mekeor: right
	<mekeor> alright.

Unless you have a different opinion, I'm going to submit a patch next week which implements this dependency.