Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 350545 - x11-terms/xterm-267 does not install menu entries (.desktop files)
Summary: x11-terms/xterm-267 does not install menu entries (.desktop files)
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-04 14:19 UTC by Michelangelo Scopelliti
Modified: 2011-07-09 12:03 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
emerge --info (emerge-info,6.20 KB, text/plain)
2011-01-04 14:20 UTC, Michelangelo Scopelliti 		Details
ebuild patch (ebuild.patch,718 bytes, patch)
2011-01-04 14:24 UTC, Michelangelo Scopelliti 		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 Michelangelo Scopelliti 2011-01-04 14:19:08 UTC 
xterm.desktop and uxterm.desktop are not installed. Apparently, it's related to the value of DESKTOP_FLAGS in the Makefile (supposedly pointing to absolute path; wild guessing).

Reproducible: Always

Steps to Reproduce:
1. emerge xterm
2. equery f xterm
3. --> no desktop files installed
Comment 1 Michelangelo Scopelliti 2011-01-04 14:20:38 UTC 
Created attachment 258822 [details]
emerge --info

I don't think it's relevant, but here's my emerge --info
Comment 2 Michelangelo Scopelliti 2011-01-04 14:24:02 UTC 
Created attachment 258824 [details, diff]
ebuild patch

A quick and dirty workaround. Actually, I think it's easier than sedding/patching Makefile, but it's just my opinion.
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2011-07-09 12:03:21 UTC 
well I don't want to add depend of desktop-file-utils to xterm's ebuild and desktop-file-install command sucks with $DESTDIR in any case so your solution using domenu is sufficient

thanks for the patch, it's now in 270