Bug 433477 - Review Request: guake - Drop-down terminal for GNOME
Review Request: guake - Drop-down terminal for GNOME
Status: CLOSED DUPLICATE of bug 444788
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
low Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
Depends On:
  Show dependency treegraph
Reported: 2008-02-19 10:45 EST by Torsten Trautwein
Modified: 2008-04-30 12:14 EDT (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-04-29 11:06:00 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Torsten Trautwein 2008-02-19 10:45:35 EST
Spec URL: http://www.neowork.de/guake/guake.spec
SRPM URL: http://www.neowork.de/guake/guake-0.1-1.src.rpm
Description: Guake is a drop-down terminal for Gnome Desktop Environment,
so you just need to press a key to invoke him,
and press again to hide.

Note: This is my first package, so I'm seeking a sponsor. Thanks.

guake.src:11: W: buildprereq-use python-devel, glib2-devel >= 2.0, libX11-devel, notify-python
guake.src: W: strange-permission guake-0.1.tar.gz 0744
guake.src: W: strange-permission guake.spec 0744
Comment 1 Guidolin Francesco 2008-02-19 14:59:05 EST
Hi, this is an unofficial review.

There are some things that need to be fixed:

[-] You should use %{dist} in the release tag:
Release:  1%{dist}
[-] You should use %{?_smp_mflags} in the make command:
make %{?_smp_mflags}
this enable parallel build on multicore CPU.
[-] Use %{_sysconfdir} instead of /etc:
Comment 2 Torsten Trautwein 2008-02-19 16:55:19 EST

thanks for your review. As this is my first rpm package, I really appreciate any

I fixed the the mistakes you mentioned. The new version is available here:

Comment 3 Guidolin Francesco 2008-02-20 10:46:15 EST
Ok, there are some other things:

[-] The recommended value for the BuildRoot tag is:
BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

[-] Packages should not use the PreReq tag.

[-] *.la file should be removed. Add:
find %{buildroot} -name \*\.la -delete

after make install and remove those file from the file list.


Comment 4 Mamoru TASAKA 2008-03-20 14:16:42 EDT
Comment 5 Mamoru TASAKA 2008-04-03 11:57:01 EDT
ping again?
Comment 6 Mamoru TASAKA 2008-04-16 14:23:11 EDT
This review request will be closed if no response from the reporter is
received within ONE WEEK.
Comment 7 Mamoru TASAKA 2008-04-29 11:06:00 EDT
Once closing.

If someone wants to import this package into Fedora, please file a new
review request and mark this bug a duplicate of the new one.

Thank you!
Comment 8 Darryl L. Pierce 2008-04-30 12:14:51 EDT

*** This bug has been marked as a duplicate of 444788 ***

Note You need to log in before you can comment on or make changes to this bug.