Bug 553739 - Review Request: kbackup - Back up your data in a simple, user friendly way
Summary: Review Request: kbackup - Back up your data in a simple, user friendly way
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Mamoru TASAKA
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-01-08 19:26 UTC by Alain Portal
Modified: 2010-05-31 19:16 UTC (History)
6 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2010-01-14 20:26:03 UTC
Type: ---
Embargoed:
mtasaka: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Alain Portal 2010-01-08 19:26:41 UTC
Spec URL: http://dionysos.fedorapeople.org/SPECS/kbackup.spec
SRPM URL: http://dionysos.fedorapeople.org/SRPMS/kde-plasma-birthday-reminder-0.9.72-1.fc12.src.rpm
Description: KBackup is a program that lets you back up any directories or files,
whereby it uses an easy to use directory tree to select the things to back up.
The program was designed to be very simple in its use
so that it can be used by non-computer experts.
The storage format is the well known TAR format, whereby the data
is still stored in compressed format (bzip2 or gzip).

Comment 1 Andrea Musuruane 2010-01-08 19:49:23 UTC
Wrong SRPMS.

Comment 2 Kevin Kofler 2010-01-08 20:47:06 UTC
The correct SRPM is: http://dionysos.fedorapeople.org/SRPMS/kbackup-0.6.3-1.fc12.src.rpm

Comment 3 Alain Portal 2010-01-08 21:20:53 UTC
Sorry...
Thank Kevin.

Comment 4 Mamoru TASAKA 2010-01-09 17:47:33 UTC
Well,

* Desktop file installation
  - "Application" in Categories is deprecated and should be removed.

  - By the way, as you already apply a patch for desktop file,
    I don't see any necessity to use "--add-category" or "--remove-category"
    and I think merging these options to the patch is much simpler.

* Scriptlets
  - As this package installs XML file in %{_datadir}/mime/packages,
    please refer to
    https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#mimeinfo

  - GTK icon related scriptlets are updated. Please refer to:
    https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache

Comment 5 Alain Portal 2010-01-09 19:32:20 UTC
(In reply to comment #4)
> Well,
> 
> * Desktop file installation
>   - "Application" in Categories is deprecated and should be removed.

OK, done.

>   - By the way, as you already apply a patch for desktop file,
>     I don't see any necessity to use "--add-category" or "--remove-category"
>     and I think merging these options to the patch is much simpler.

In fact, I just made this patch three hours ago to add the X-DocPath entry because help wasn't listed in the khelpcenter and to reorganize a little this file without removing categories provided by upstream.
So, if upstream accept my patch, I'll still need to use "--add-category" or "--remove-category" for future release, so, I wish to keep them if you agree.

> * Scriptlets
>   - As this package installs XML file in %{_datadir}/mime/packages,
>     please refer to
>     https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#mimeinfo
> 
>   - GTK icon related scriptlets are updated. Please refer to:
>     https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache    

OK, done, thanks for the reminder.

%changelog
* Sat Jan  9 2010 Alain Portal <alain.portal[AT]univ-montp2[DOT]fr> 0.6.3-2
  - Don't add Application category in desktop file
  - Add update-mime-database in scriptlets
  - Update gtk-update-icon-cache use in scriptlets
  - Add %%posttrans scriptlet

News files:
http://dionysos.fedorapeople.org/SPECS/kbackup.spec
http://dionysos.fedorapeople.org/SRPMS/kbackup-0.6.3-2.fc12.src.rpm

Comment 6 Mamoru TASAKA 2010-01-10 16:24:24 UTC
One point.
- Now we usually use %defattr(-,root,root,-)

------------------------------------------------------------
    This package (kbackup) is APPROVED by mtasaka
------------------------------------------------------------

Comment 7 Alain Portal 2010-01-10 16:59:01 UTC
(In reply to comment #6)
> One point.
> - Now we usually use %defattr(-,root,root,-)

OK, done.

%changelog
* Sun Jan 10 2010 Alain Portal <alain.portal[AT]univ-montp2[DOT]fr> 0.6.3-3
  - Updated %%defattr macro

News files:
http://dionysos.fedorapeople.org/SPECS/kbackup.spec
http://dionysos.fedorapeople.org/SRPMS/kbackup-0.6.3-3.fc12.src.rpm

> ------------------------------------------------------------
>     This package (kbackup) is APPROVED by mtasaka
> ------------------------------------------------------------    

Am I allowed to commit the package or do I have to wait something?

Comment 8 Alain Portal 2010-01-10 17:02:47 UTC
Oups...
Thanks for the review ;-)

Comment 9 Mamoru TASAKA 2010-01-10 17:34:40 UTC
(In reply to comment #7)
Please follow
http://fedoraproject.org/wiki/New_package_process_for_existing_contributors
from "Step 7" (i.e. first write CVS request as written in)
http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

Please mention in the CVS request that this package was once got orphaned 
and this is to reintroduce the existed package.

Comment 10 Alain Portal 2010-01-10 17:47:27 UTC
OK, thanks.

Comment 11 Alain Portal 2010-01-10 17:58:18 UTC
New Package CVS Request
=======================
Package Name: kbackup
Short Description: Back up your data in a simple, user friendly way
Owners: dionysos
Branches: F-11 F-12
InitialCC:

This package was once got orphaned and this is to reintroduce the existed package.

Comment 12 Jason Tibbitts 2010-01-13 20:41:31 UTC
This package already exists, so we can't do a "new package CVS request".  What I've done is create the F-11 and F-12 branches.  If there was something else you needed, please re-raise the fedora-cvs flag.

Comment 13 Mamoru TASAKA 2010-01-14 20:26:03 UTC
Closing (rebuild done for all branches, push request submitted
on bodhi for F-12/11)

Comment 14 Alain Portal 2010-05-31 18:49:48 UTC
Package Change Request
======================
Package Name: kbackup
New Branches: EL-6

Comment 15 Kevin Fenzi 2010-05-31 19:16:36 UTC
cvs done.


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