bugDenemo - Bugs: bug #26221, OpenSuse Packaging Errors -...

 
 

bug #26221: OpenSuse Packaging Errors - Denemos goes against FreeDesktop rules.

Submitted by:  -Deleted Account- <steele>
Submitted on:  Sat 18 Apr 2009 06:59:16 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: Jeremiah Benham <jjbenham>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Mon 20 Apr 2009 10:17:52 PM UTC, comment #4:

Ok thanks, I got the current git version to compile in OBS.

But there has been a misunderstanding it seems, about the first point I made in this report: it isn't necessary to rename the denemo.png file to "denemo". I only wanted to point out that the freedesktop standard says that the "Icon" entry in the .desktop file should only contain "denemo", not "denemo.png".

This might even be a non issue because it works either way in practice, it seems .

So I suggest to rename the icon file to "denemo.png" but leave the desktop "Icon" entry as "denemo".

Installing to /usr/share/pixmaps fixed the issue I was having.

Sorry for the confusion, I hope to have cleared it up now...

Anonymous
Mon 20 Apr 2009 02:00:35 PM UTC, comment #3:

I made some changes to the git version. Can you tell me if this has fixed it or not? You will probably need to run ./autogen.sh and all that first.

Jeremiah Benham <jjbenham>
Project AdministratorIn charge of this item.
Sun 19 Apr 2009 04:44:06 PM UTC, comment #2:

yes, /usr/share/pixmaps is what i meant, sorry...

To clarify: The problem appears only if I build inside the build service, which uses a chroot environment, rpmlint, suse specific quality tests and other checks to make a build safe against common errors, security risks and so forth. The error I was getting was somewhat confusing but it said that something was wrong with the .desktop file. So I tried a few things, and as soon as I did this in the denemo.spec file:

%install
...
%__mv "$RPM_BUILD_ROOT"/%{_datadir}/icons "$RPM_BUILD_ROOT"/%{_datadir}/pixmaps

%files
...
# %{_datadir}/icons/*
%{_datadir}/pixmaps/*
...

the "error" was gone.

The build still fails for reasons I'll post in another ticket later....

Anonymous
Sat 18 Apr 2009 09:55:51 AM UTC, comment #1:

I think you mean /usr/share/pixmaps not /usr/src/pixmaps.

Otherwise I agree with your 2 points, but as denemo builds and works OK in Fedora, I suspect your main problem is elsewhere.

Roy Rankin <rrankin>
Project Member
Sat 18 Apr 2009 06:59:16 AM UTC, original submission:

From Edogawa via IRC:

i'm running into a problem with building a package from the 0.8.2 tarball at openSUSE Build Service

i don't know yet if that's what causes the build to be refused, but i think so... (building locally with rpmbuild works fine but the rules at the build service are much more strict)

1) the desktop file names the icon with file extension (denemo.png) while the standard wants just the basename ("denemo")

2) (worse) the icon gets installed to /usr/share/icons while it should be in /usr/src/pixmaps (according to the freedesktop standard again)

see

http://forgeftp.novell.com/library/SUSE%20Package%20Conventions/spc_desktop_menu.html

, 9.2 a bit down under optional keys/Icon

-Deleted Account- <steele>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jjbenham (Updated the item)
  • -unavailable- added by rrankin (Posted a comment)
  • -unavailable- added by steele (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 02 Mar 2011 09:55:03 PM UTCjjbenhamOpen/ClosedOpen=>Closed
    Sat 18 Apr 2009 12:29:44 PM UTCjjbenhamAssigned toNone=>jjbenham

    Back to the top


    Powered by Savane 3.1-cleanup1