Bug 195677 - [PATCH] devel/xdg-utils: xdg-open broken for non-KDE/non-GNOME users
Summary: [PATCH] devel/xdg-utils: xdg-open broken for non-KDE/non-GNOME users
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-gnome (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-12-04 14:04 UTC by Ulrich Spörlein
Modified: 2016-01-17 11:18 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (gnome)

Attachments
patch for devel/xdg-utils (1.01 KB, patch)
2014-12-04 14:04 UTC, Ulrich Spörlein 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ulrich Spörlein freebsd_committer freebsd_triage 2014-12-04 14:04:38 UTC 
Created attachment 150186 [details]
patch for devel/xdg-utils

xdg-open falls through to some code for the non-KDE, non-GNOME case that relies on the built-in printf to turn things like \x20 into a literal space. This was fixed in Debian by using their /usr/bin/printf, which does that. Ours doesn't but Jilles pointed me to unvis(1) which can decode URIs just fine (yay!).

See also:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=612339
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=613272
Comment 1 Bugzilla Automation freebsd_committer freebsd_triage 2014-12-04 14:04:38 UTC 
Auto-assigned to maintainer gnome@FreeBSD.org
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-01-17 11:13:07 UTC 
A commit references this bug:

Author: kwm
Date: Sun Jan 17 11:12:17 UTC 2016
New revision: 406300
URL: https://svnweb.freebsd.org/changeset/ports/406300

Log:
  xdg-open falls through to some code for the non-KDE, non-GNOME case that
  relies on the built-in printf to turn things like \x20 into a literal
  space. This was fixed in Debian by using their /usr/bin/printf, which
  does that. Ours doesn't but Jilles pointed me to unvis(1) which can
  decode URIs just fine (yay!).

  PR:		195677
  Submitted by:	uqs@

Changes:
  head/devel/xdg-utils/Makefile
  head/devel/xdg-utils/files/patch-scripts_xdg-open
Comment 3 Koop Mast freebsd_committer freebsd_triage 2016-01-17 11:18:29 UTC 
Committed, thanks.