[Bug 458338] Review Request: DivFix++ - A program to repair broken AVI file streams by rebuilding index part of file

bugzilla at redhat.com bugzilla at redhat.com
Fri Oct 10 19:22:10 UTC 2008


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=458338





--- Comment #8 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp>  2008-10-10 15:22:08 EDT ---
(In reply to comment #7)
First of all:
> > > Hmmm. What about this
> > > http://fedoraproject.org/wiki/PackageMaintainers/Packaging_Tricks#Avoiding_using_fedora_or_redhat
  - This is packaging tricks (as the title says) and not guidelines.

> So, if you oppose, I think you should discuss about it with FESCO...
  - So no need.

    On the contrary, see this:
https://fedoraproject.org/wiki/Packaging/Guidelines#desktop-file-install_usage
----------------------------------------------------------------
If upstream uses <vendor_id>, leave it intact, otherwise use fedora as
<vendor_id>
----------------------------------------------------------------
  But as I said before not a few maintainers simply removes --vendor=foo.

> > * Category of desktop file
> >   - As you create the base desktop file by yourself, you can simply
> >     add 
> > ------------------------------------------------
> > Category=Video;
> > ------------------------------------------------
> >     line between "%{__cat} > %{name}.desktop << EOF"
> >     and EOF lines, then remove "--add-category=Video"
> >     ! Note
> >       - Semicolon is needed at the last.
> Ok, I'm do how you say.
> Is there any differences where Category mentioned?
> 
> And I tryed this, but got error:
> /var/tmp/DivFix++-0.30-3.fc9-root-pasha//usr/share/applications/DivFix++.desktop:
> error: file contains key "Category" in group "Desktop Entry", but keys
> extending the format should start with "X-"
> Error on file "DivFix++.desktop": Failed to validate the created desktop file

- Because the correct one is "Categories=Video;", sorry...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list