Bug 22146 - LXQt: pcmanfm-qt, lxqt-sudo, lxqt-admin
Summary: LXQt: pcmanfm-qt, lxqt-sudo, lxqt-admin
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Florian Hubold
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-12-08 19:13 CET by Dimitrios Glentadakis
Modified: 2017-12-14 19:43 CET (History)
4 users (show)

See Also:
Source RPM:
CVE:
Status comment:


Attachments

Description Dimitrios Glentadakis 2017-12-08 19:13:59 CET
I have updated the following specs:

lxqt-sudo
Imported
http://svnweb.mageia.org/packages/cauldron/lxqt-sudo/current/SPECS/lxqt-sudo.spec?view=log

lxqt-admin
Imported
http://svnweb.mageia.org/packages/cauldron/lxqt-admin/current/SPECS/lxqt-admin.spec?view=log

pcmanfm-qt
Update to 0.12.0
Replace kdesudo with lxqt-sudo
http://svnweb.mageia.org/packages/cauldron/pcmanfm-qt/current/SPECS/pcmanfm-qt.spec?view=log

task-lxqt
- Add lxqt-sudo in task-lxqt-minimal Requires
- Add lxqt-admin in task-lxqt Recommends
- Add sddm-config-editor in task-lxqt Recommends
http://svnweb.mageia.org/packages/cauldron/task-lxqt/current/SPECS/task-lxqt.spec?view=log
Dimitrios Glentadakis 2017-12-08 19:15:05 CET

CC: (none) => doktor5000, geiger.david68210, mageia, marja11

Comment 1 Marja Van Waes 2017-12-10 09:41:01 CET
@ Dimitrios

Thanks for not pushing your changes to pcmanfm-qt and talk-lxqt, along with your new packages, without contacting the maintainer of pcmanfm-qt and task-lxqt and recent committers/pushers of lxqt*

Assigning to doktor5000, because he is the registered maintainer of pcmanfm-qt and talk-lxqt

Cheers,
Marja

Assignee: bugsquad => doktor5000

Comment 2 Florian Hubold 2017-12-10 14:53:39 CET
(In reply to Dimitrios Glentadakis from comment #0)
> I have updated the following specs:

Hi Dimitrios, greatly appreciated. Unfortunately I don't find the time anymore to maintain all the lxqt packages, so as you're using them regularly, shall I release them so you can take over maintainership?

Apart from that, for the commit to pcmanfm-qt it seems you also did a reindentation (tabs to spaces or spaces to tabs) - you should keep such things in separate commits away from other changes, and always indicate that this has been done as otherwise it might be strange for others to see such a huge diff just for a version bump ;)
Comment 3 Dimitrios Glentadakis 2017-12-12 05:23:40 CET
Ok Florian, i can continue the maintainership among others of course.

Thanks for the suggestion for the separated commits for the diffs, i will pay attention on this.
Comment 4 Dimitrios Glentadakis 2017-12-14 19:43:08 CET
The packages have been uploaded

Resolution: (none) => FIXED
Status: NEW => RESOLVED


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