Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
Summary: Review Request: cachefilesd Product: Fedora Core Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: notting@redhat.com ReportedBy: steved@redhat.com QAContact: extras-qa@fedoraproject.org CC: fedora-package-review@redhat.com
Spec URL: http://people.redhat.com/steved/fscache/cachefilesd/0.3-2/cachefilesd.spec
SRPM URL: http://people.redhat.com/steved/fscache/cachefilesd/0.3-2/cachefilesd-0.3-2....
Description: The cachefilesd daemon manages the caching files and directory that are that are used by network filesystems such a AFS and NFS to do persistent caching to the local disk.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO_REPORTER AssignedTo|notting@redhat.com |jkeating@redhat.com OtherBugsDependingO|188265 |188267 nThis| |
------- Additional Comments From jkeating@redhat.com 2006-07-28 12:49 EST ------- NEEDSWORK: - no URL tag - Source0 should be a full URL if possible - Missing BuildRequires, at least automake - Requires(post): /usr/bin/chkconfig - Requires(preun): /usr/bin/chkconfig - Init scripts aren't often marked as config files...
Please fix up the missing build requires (test by 'brew build --scratch --arch-override x86_64 dist-5E cachefilesd-0.3-2.src.rpm' and 'brew watch-logs <failedtaskid>) and I'll build and run rpmlint on it.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
steved@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |ASSIGNED
------- Additional Comments From steved@redhat.com 2006-07-28 15:23 EST ------- - no URL tag - Source0 should be a full URL if possible There is no upstream at this point so there is URL at the moment. I guess I could through things it on my people pages if that would be ok...
- Missing BuildRequires, at least automake - Requires(post): /usr/bin/chkconfig - Requires(preun): /usr/bin/chkconfig Done.
- Init scripts aren't often marked as config files... The init script is in the tarball. So do I need to break it out and create a file called cachefiles.init?
Please fix up the missing build requires (test by 'brew build --scratch --arch-override x86_64 dist-5E cachefilesd-0.3-2.src.rpm' and 'brew watch-logs <failedtaskid>)
Done... see taskID 138115
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO_REPORTER
------- Additional Comments From jkeating@redhat.com 2006-07-31 14:42 EST ------- er, the init script thing, just don't mark it as %config in the spec file.
Pointing to your people page is fine, or even just www.redhat.com since this is a Red Hat project.
I'll await an updates .spec and .srpm.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
steved@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |ASSIGNED
------- Additional Comments From steved@redhat.com 2006-08-01 13:44 EST ------- The following changes were made to the spec file and srpm in: http://people.redhat.com/steved/fscache/cachefilesd/0.3-2
diff -r1.8 cachefilesd.spec 3c3 < Release: 2%{?dist} ---
Release: 3%{?dist}
8c8 < Source0: cachefilesd-0.3.tar.bz2 ---
Source0:
http://people.redhat.com/dhowells/cachefilesd/cachefilesd-0.3.tar.bz2 71c71 < %config %{_sysconfdir}/rc.d/init.d/cachefilesd ---
%{_sysconfdir}/rc.d/init.d/cachefilesd
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO_REPORTER
------- Additional Comments From jkeating@redhat.com 2006-08-01 16:22 EST ------- rpmlint is warning you that there is no standalone URL tag which might point to a project page.
Your spec /srpm wasn't updated at your sight, but I'm applying the patches to it by hand.
Hrm, somehow I missed this before: there is no actual cleaning of rpmbuildroot in %clean, please fix that.
E: cachefilesd zero-length /usr/share/doc/cachefilesd-0.3/ChangeLog
And would it kill ya to add URL: http://www.redhat.com
or something similar?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
steved@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |ASSIGNED
------- Additional Comments From steved@redhat.com 2006-08-01 16:35 EST ------- err... http://people.redhat.com/steved/fscache/cachefilesd/0.3-3 has the correct spec and srpm...
adding a http://www.redhat.com URL would be fine...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
------- Additional Comments From jkeating@redhat.com 2006-08-01 17:34 EST ------- Still doesn't have the changes I mentioned above...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
------- Additional Comments From steved@redhat.com 2006-08-01 21:54 EST ------- Maded the following changed to the spec that can be found in http://people.redhat.com/steved/fscache/cachefilesd/0.3-3
diff -r1.9 cachefilesd.spec 6a7,8
56a59
rm -rf $RPM_BUILD_ROOT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO_REPORTER OtherBugsDependingO|188267 |188268 nThis| |
------- Additional Comments From jkeating@redhat.com 2006-08-02 11:40 EST ------- Ok, with that this package is approved.
Will this need to be on comps somewhere, or would you be marking this as a dep of some other package?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
steved@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |ASSIGNED
------- Additional Comments From steved@redhat.com 2006-08-03 09:30 EST ------- No and no...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO_REPORTER
------- Additional Comments From jkeating@redhat.com 2006-08-04 08:44 EST ------- I've added cachefilesd to dist-fc6 as owned by steved. It is too late for Test2, but if this is a blocker for RHEL5 B1 you can get it included there.
I'm a bit confused, if its not a Requires of something, nor in Comps anyplace, how will a user actually install it?
Please close this bug when you've built it into dist-fc6.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
jkeating@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO_REPORTER |CLOSED Resolution| |RAWHIDE
------- Additional Comments From jkeating@redhat.com 2006-08-07 15:26 EST ------- Built into dist-fc6, still has a question on why not in comps.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
------- Additional Comments From dhowells@redhat.com 2006-08-08 05:06 EST -------
I'm a bit confused, if its not a Requires of something, nor in Comps anyplace, how will a user actually install it?
That means we need to update the installer?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=200551
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium Priority|normal |medium Product|Fedora Core |Fedora
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/show_bug.cgi?id=200551
bugzilla@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Version|devel |rawhide
coughlan@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |coughlan@redhat.com Flag| |fedora-cvs?
------- Additional Comments From coughlan@redhat.com 2008-03-13 15:23 EST ------- Package Change Request ====================== Package Name: cachefilesd [New Branches: ] [Updated Fedora Owners: ] dhowells@redhat.com [Updated Fedora CC: ] [Updated EPEL Owners: ] [Updated EPEL CC: ] [Updated Description: ] [Updated Cvsextras Commits: ] [add any required explanatory text here] David is the primary author.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report.
Summary: Review Request: cachefilesd
https://bugzilla.redhat.com/show_bug.cgi?id=200551
kevin@tummy.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kevin@tummy.com Flag|fedora-cvs? |
------- Additional Comments From kevin@tummy.com 2008-03-13 19:57 EST ------- Is steve ok with handing off the package maintaining?
Is dhowells ok with taking over the package maintaining? I'd be happy to make the changes with their acks... or they can go to:
https://admin.fedoraproject.org/pkgdb/packages/name/cachefilesd
and make the changes themselves.
package-review@lists.fedoraproject.org