Bug 1158999 (lxqt-session)

Summary: lxqt-session - Main session for LXQt desktop suite
Product: [Fedora] Fedora Reporter: Helio Chissini de Castro <hcastro>
Component: Package ReviewAssignee: Rex Dieter <rdieter>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: jmunilla, package-review, rdieter, ti.eugene
Target Milestone: ---Flags: rdieter: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-11-20 13:10:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1157402, 1158632    
Bug Blocks:    

Description Helio Chissini de Castro 2014-10-30 16:30:57 UTC
Spec URL: https://heliocastro.fedorapeople.org/lxqt/lxqt-session.spec
SRPM URL: https://heliocastro.fedorapeople.org/lxqt/lxqt-session-0.8.0-2.fc21.src.rpm
Description: Main session for LXQt desktop suite
Fedora Account System Username:

Comment 1 Rex Dieter 2014-11-07 17:37:48 UTC
naming: ok

sources: ok
d1bdb44afd29974ac9362e9384296495  lxqt-session-0.8.0.tar.xz

macros: ok

scriptlets: ok, n/a

1. license: ok, but could be ammended to
License: LGPLv2+

2.  SHOULD omit
Obsoletes: razorqt-session <= 0.5.2
for now, until you and razorqt maintainers agree on a migration plan (if any).  There are many other razorqt-related packages that need consideration wrt Obsoletes

3.  SHOULD add
Requires: xdg-utils
since I see build flag, -DBUNDLE_XDG_UTILS=NO, I'd venture it may want it.


Otherwise, it's small, simple, clean.


APPROVED

Comment 3 Rex Dieter 2014-11-08 13:01:48 UTC
<nod>, adding .desktop validation here is needed.

directory ownership is problem of liblxqt (fixed there now).

Comment 5 Rex Dieter 2014-11-10 15:09:02 UTC
Thanks, even better, APPROVED++

Comment 6 Helio Chissini de Castro 2014-11-12 15:01:30 UTC
New Package SCM Request
=======================
Package Name: lxqt-session
Short Description: Main session for LXQt desktop suite
Upstream URL: http://lxqt.org/downloads/lxqt/0.8.0/lxqt-session-0.8.0.tar.xz
Owners: heliocastro rdieter tieugene
Branches: f20 f21 el6 epel7
InitialCC: heliocastro

Comment 7 Rex Dieter 2014-11-13 17:54:05 UTC
New Package SCM Request
=======================
Package Name: lxqt-session
Short Description: Main session for LXQt desktop suite
Upstream URL: http://lxqt.org/
Owners: heliocastro rdieter tieugene
Branches: f20 f21 el6 epel7
InitialCC:

Comment 8 Gwyn Ciesla 2014-11-13 19:41:04 UTC
Git done (by process-git-requests).

Comment 9 Rex Dieter 2014-11-20 13:10:32 UTC
imported into rawhide, holding off on builds for now