Bug 912878 - Review Request: profile-sync-daemon - Offload browser profiles to RAM for speed a wear reduction
Summary: Review Request: profile-sync-daemon - Offload browser profiles to RAM for spe...
Keywords:
Status: CLOSED DUPLICATE of bug 968253
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-02-19 21:52 UTC by graysky
Modified: 2013-10-19 14:42 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-05-29 09:16:48 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description graysky 2013-02-19 21:52:15 UTC
Spec URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon.spec
SRPM URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.24-1.fc18.src.rpm
Description: Symlinks and syncs browser profiles to RAM via tmpfs which will reduce HDD/SDD calls and speed-up browsers.
Fedora Account System Username: graysky

Comment: This is my first package, and I am seeking a sponsor. I am the upsteam dev of this software and have packaged it for: Arch Linux, Gentoo, and Ubuntu. I would like to extend that to include fc18 as well.

Comment 1 graysky 2013-02-20 20:40:50 UTC
After trying for several days to get the web interface for koji to work (still cannot login), I just build via CLI:

http://koji.fedoraproject.org/koji/taskinfo?taskID=5037743

Comment 2 Miro Hrončok 2013-02-20 23:26:57 UTC
You probably shouldn't copy upstream changelog to %changelog, it's simply a changelog of the spec/SRPM.

* Tue Feb 19 2013 graysky <graysky> - 5.24-1
- New release 5.24

Should be fine.

On the other hand, be sure to add the CHANGELOG and README.md to %doc. Once approved, don't forget to update the Fedora package information in README.md, so the offical package doesn't point users to the unofficial (by official, I mean in Fedora repos).

Why there is a Makefile.fedora18 file, when you use Makefile.archlinux? They are identical, but anyway, doesn't make much sense to me.

Comment 3 graysky 2013-02-21 00:12:57 UTC
(In reply to comment #2)
> You probably shouldn't copy upstream changelog to %changelog, it's simply a
> changelog of the spec/SRPM.
> 
> * Tue Feb 19 2013 graysky <graysky> - 5.24-1
> - New release 5.24
> 
> Should be fine.
> 
> On the other hand, be sure to add the CHANGELOG and README.md to %doc. Once
> approved, don't forget to update the Fedora package information in
> README.md, so the offical package doesn't point users to the unofficial (by
> official, I mean in Fedora repos).

Took your suggestions with one exception: should I really include the github README.md in the package?

Files:
http://repo-ck.com/fedora_rpms/profile-sync-daemon.spec
http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.24-2.fc18.src.rpm
http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.24-2.fc18.noarch.rpm

Koji build log:
http://koji.fedoraproject.org/koji/taskinfo?taskID=5038248

> Why there is a Makefile.fedora18 file, when you use Makefile.archlinux? They
> are identical, but anyway, doesn't make much sense to me.

Ah, my bad.  Fixed in 5.24-2 available at the URL above.

Comment 4 graysky 2013-02-23 11:12:00 UTC
I updated the upstream code and refreshed the respective files for review:

Spec URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon.spec
SRPM URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.26-1.fc18.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5046536

Is there anything I can do to help expedite the review process?

Comment 5 graysky 2013-03-14 19:49:28 UTC
I have updated the upstream code and refreshed the respective files for review:

Spec URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon.spec
SRPM URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.29-1.fc18.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5122938

Comment 6 graysky 2013-03-14 19:52:39 UTC
Opps... correct koji task: http://koji.fedoraproject.org/koji/taskinfo?taskID=5122938

Comment 7 graysky 2013-05-29 04:23:20 UTC
I have updated the upstream code and refreshed the respective files for review:

Spec URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon.spec
SRPM URL: http://repo-ck.com/fedora_rpms/profile-sync-daemon-5.35-1.fc18.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=5436525

Comment 8 graysky 2013-05-29 09:16:48 UTC
Closing report per Christopher's request.

Comment 9 Christopher Meng 2013-05-29 10:15:53 UTC

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


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