This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 452324 - Review Request: lxterminal - Desktop-independent VTE-based terminal emulator
Review Request: lxterminal - Desktop-independent VTE-based terminal emulator
Status: CLOSED NEXTRELEASE
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Miroslav Lichvar
Fedora Extras Quality Assurance
:
Depends On:
Blocks: LXDE
  Show dependency treegraph
 
Reported: 2008-06-20 19:08 EDT by Christoph Wickert
Modified: 2009-06-13 15:27 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-07-04 18:16:32 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
mlichvar: fedora‑review+
kevin: fedora‑cvs+


Attachments (Terms of Use)

  None (edit)
Description Christoph Wickert 2008-06-20 19:08:00 EDT
Spec URL: http://cwickert.fedorapeople.org/review/lxterminal-0.1.2-1.fc10.src.rpm
SRPM URL: http://cwickert.fedorapeople.org/review/lxterminal.spec
Description: 
LXterminal is a VTE-based terminal emulator with support for multiple tabs. 
It is completely desktop-independent and does not have any unnecessary 
dependencies. In order to reduce memory usage and increase the performance 
all instances of the terminal are sharing a single process.
Comment 1 Christoph Wickert 2008-07-03 12:34:37 EDT
Miroslav, please wait a moment, I'm going to upload 0.1.3. 
Comment 2 Christoph Wickert 2008-07-03 12:37:59 EDT
New SRPM: http://cwickert.fedorapeople.org/review/lxterminal-0.1.3-1.fc10.src.rpm
Comment 3 Miroslav Lichvar 2008-07-03 13:35:41 EDT
The spec file is cleanly written, everything seems good.

- rpmlint is silent
- the package is named according to the Package Naming Guidelines
- the spec file name matches the base package %{name}
- the package meets the Packaging Guidelines
- the package is licensed with a Fedora approved license
- the License field in the package spec file matches the actual license (GPLv2+)
- the package includes the text of the license in %doc
- the spec file is written in American English
- the spec file for the package is legible
- the sources used to build the package matches the upstream source
- the package successfully builds in x86_64 and ppc rawhide
- all build dependencies are listed in BuildRequires
- the spec file handles locales properly
- the package owns all directories that it creates
- the package does not contain any duplicate files in the %files listing
- permissions on files are set properly
- the package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT
- the package consistently uses macros
- the package contains code, or permissible content
- files included as %doc don't affect the runtime of the application
- desktop file is installed with desktop-file-install
- the package does not own files or directories already owned by other packages
- at the beginning of %install, the package runs rm -rf $RPM_BUILD_ROOT
- all filenames in rpm packages are valid UTF-8
- the package functions as described (tested only on x86_64)

Approved.

The sed line in %prep can be removed, the typo was fixed in 0.1.3.
Comment 4 Christoph Wickert 2008-07-03 14:10:23 EDT
(In reply to comment #3)
> The sed line in %prep can be removed, the typo was fixed in 0.1.3.

Good catch (although I should have known that because I was the one who informed
upstream about the typo). Will fix that after the import.

Thanks for the review!

New Package CVS Request
=======================
Package Name: lxterminal
Short Description: Desktop-independent VTE-based terminal emulator
Owners: cwickert
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes
Comment 5 Kevin Fenzi 2008-07-04 16:14:13 EDT
cvs done.
Comment 6 Fedora Update System 2008-07-04 18:14:24 EDT
lxterminal-0.1.3-1.fc9 has been submitted as an update for Fedora 9
Comment 7 Fedora Update System 2008-07-04 18:15:24 EDT
lxterminal-0.1.3-1.fc8 has been submitted as an update for Fedora 8
Comment 8 Fedora Update System 2008-07-06 02:12:07 EDT
lxterminal-0.1.3-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 9 Fedora Update System 2008-07-06 02:17:12 EDT
lxterminal-0.1.3-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 10 Fedora Update System 2008-07-26 01:56:31 EDT
lxterminal-0.1.3-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.
Comment 11 Christoph Wickert 2009-05-22 21:45:37 EDT
Package Change Request
======================
Package Name: lxterminal
New Branches: EL-4 EL-5
Owners: cwickert
Comment 12 Kevin Fenzi 2009-05-23 01:27:10 EDT
cvs done.

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