[Bug 680268] Review Request: cputnik - Dockapp which displays cpu and memory usage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=680268

--- Comment #5 from Martin Gieseking <martin.gieseking@xxxxxx> 2011-02-25 15:32:34 EST ---
(In reply to comment #3)
> cd is required here, because the Makefile resides in /src.

There's also a Makefile in the base directory that calls make for src/.
However, both variants work here anyway.
But please shorten the install line as mentioned in comment #2. If you build
inside of src/ your probably have to prepend "src/" to %{name}.

> I have changed it. However, the modern Âapps as known from the Âapp storesÂ
> are female. But ÂDockapp means ÂDockable appletÂ, and this is not female.

Ah, right. Now that you mention it, I'm convinced. :)

> Grrr, in German we have thirteen cases of the English ÂtheÂ, it's sometimes
> difficult...

Yes, indeed. Especially, if you have to choose a gender for foreign words...


(In reply to comment #4)
> Another question: A dockapp is a graphical application. That's why I have
> actually to add a *.desktop file to make it available from the menus. 

According to the guidelines, only GUI applications require a .desktop file.
Dockapps don't fall into this category to me. The following old thread seems to
confirm that:
http://www.redhat.com/archives/fedora-extras-list/2006-October/msg00332.html

Maybe you should ask on the packaging mailing list to get a reconfirmation.

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]