[Bug 250747] Review Request: Linux-Complete-Backup-and-Recovery-HOWTO - bare metal recovery scripts & docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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: Linux-Complete-Backup-and-Recovery-HOWTO - bare metal recovery scripts & docs


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250747





------- Additional Comments From charlescurley@xxxxxxxxxxxxxxxxx  2007-08-14 14:58 EST -------
Thank you, Mr. Boyle.

> ? - Package meets naming and packaging guidelines
> --> Seems more a set of scripts than documentation... although I could be
> looking at what it does all wrong :P  Consider renaming after script-suite?

Hmmm. The current name is the name of the document at the Linux Documentation Project, and I figured that consistency would be a good idea. The main package is some of the scripts (more are to come), and the -doc subpackage is the original HOWTO in various formats. I could make the docs the main package and put the scripts into a -scripts subpackage, I suppose, but the current naming and subpackaging are consistent with current Fedora usage. I'm open to suggestions here.


> OK? - Spec has consistant macro usage.
> --> $RPM_BUILD_ROOT could be macro-ified as %{buildroot}

OK, done. It should show up in the next version.

> NOT OK - Sources SHOULD match upstream md5sum
> --> Source: e44ce87defb0b7f3688dbbded79bedc4 Package:
> e44ce87defb0b7f3688dbbded79bedc4

I'm not sure where that came from. In any case, it should go away as soon as I put new packages up on my server.


> ? - Changelog section is correct. 
> --> Not sure if one should put such direct references to the specfile in
> %changelog... but it's probably bad form to change the %changelog after the fact... 

That was the reason for the new package version, and the sole change. As with this one.

Again, new SRPM: http://www.charlescurley.com/Linux-Complete-Backup-and-Recovery-HOWTO/srpms/Linux-Complete-Backup-and-Recovery-HOWTO-2.3-3.fc7.src.rpm

And the revised spec is at the usual: http://www.charlescurley.com/Linux-Complete-Backup-and-Recovery-HOWTO/Linux-Complete-Backup-and-Recovery-HOWTO.spec

BTW, since my web page is pretty much automated, and comments for this bugzilla entry are not, the web page (http://www.charlescurley.com/Linux-Complete-Backup-and-Recovery-HOWTO.html) is authoritative for the most recent version, not bugzilla.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]