Re: Proof reading
Bas van den Dikkenberg wrote:
> I need a little bit of help i am dislectic and need some help checking
> the spelling and grammer is ther some that want to help me.
Sure! It's no trouble.
> its aboud these 3 files, for the comments also.
Okay.
> # lintian gives a waring becouse of file premisions witch is incorrect, file premission are for security reasons
# lintian gives a warning because of file permissions which is incorrect; file permissions are for security reasons
But lintian override comments are always about unnecessary lintian
warnings, so just say:
# the "wrong" file permissions are for security reasons
> non-standard-file-perm etc/burp/burp-server.conf 0600 != 0644
> non-standard-file-perm etc/burp/burp.conf 0600 != 0644
> non-standard-dir-perm etc/burp/clientconfdir/ 0700 != 0755
> non-standard-file-perm etc/burp/ssl_cert-client.pem 0600 != 0644
> non-standard-file-perm etc/burp/ssl_cert-server.pem 0600 != 0644
> non-standard-dir-perm var/spool/burp/ 0700 != 0755
(I haven't checked these for cut-and-paste errors)
> burp (1.3.0-2) unstable; urgency=low
>
> * added forwarded header to patches
> * removed changelog entry's from patches
entries
> * removed debian/README (Closes: #659453)
> * removed blank mailto headers in cronjob and redirected out put to /var/log/burp-client (Closes: #659452)
MAILTO header output
> * added extra logrotate for /var/log/burp-client
^(excess space)
> * Change /var/run to /run (Closes: #659464)
>
> -- Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net> Sat, 11 Feb 2012 10:50:52 +0100
>
> burp (1.3.0-1) unstable; urgency=low
>
> * changed files premisions in rule file
fille permissions rules
> * removed empty todo and readme
> * Fixed an typo in the control file
a
> * override_dh_auto_configure restore is neasery to run
> ./configure without any options
> the configure files places the files in the correct
> directory voor a debian build
I assume "neasery" is "necessary" and "voor" is "for", but I don't
understand what it's saying. Something to do with a rule like this:
override_dh_auto_configure:
./configure
But what is necessary for what, and what is the change? For now I'll
assume that I would understand it if I spoke make.
> * added watch file
> * made patch to move PID file
> * Fixed lantian errors
lintian
> * Corrected the Copyright file
> * Initial release (Closes: #658152)
> * switches to 3.0 (quilt)
> * made lantian override file for file premissions
i permissions
> * added man pages for bedup and burp_ca
>
> -- Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net> Fri, 10 Feb 2012 17:14:13 +0100
(My revised version also eliminates the inconsistent capitalisation of
items and ensures they're all past tense)
> Source: burp
> Section: utils
> Priority: extra
> Maintainer: Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net>
> Build-Depends: debhelper (>= 7.0.50~), autotools-dev, libncurses5-dev, libssl-dev, zlib1g-dev, librsync-dev, libacl1-dev, libattr1-dev
> Standards-Version: 3.9.2
> Homepage: http://burp.grke.net/
> Vcs-Git: git://github.com/grke/burp.git
> #Vcs-Browser: http://git.debian.org/?p=collab-maint/burp.git;a=summary
>
> Package: burp
> Architecture: any
> Depends: ${shlibs:Depends}, ${misc:Depends}
> Description: backup and restore program
Lots of things are backup/restore systems; it would be helpful if the
synopsis gave some clue about what makes this one stand out from its
rivals. The homepage says it's like Bacula with extra features and
less complexity.
Oh, wait! I almost missed the fact it's a Why-The-Name hint. In that
case I'd be happy with just "BackUp and Restore Program", though I'd
still suggest making it:
Description: simple network BackUp and Restore Program
> Burp is a backup and restore program. It uses librsync in order to save
> the amount of space that is used by each backup. It also uses VSS (Volumer
> Shadow Copy Service) to make snapshots when backing up Windows computers.
Most importantly: s/Volumer/Volume/
The first sentence uselessly repeats the synopsis; the second
undersells the value of librsync (which also reduces the amount of
network bandwidth required); and the third confusingly expands VSS as
VSCS (though yes, Volume Snapshot Service and Volume Shadow Copy
Service are different names for more or less the same thing).
I would suggest:
Burp is a client/server backup system that saves space and network
usage by using librsync, and uses the Volume Snapshot Service when
backing up Windows clients.
--
JBR with qualifications in linguistics, experience as a Debian
sysadmin, and probably no clue about this particular package
diff -ru old/burp.lintian-overrides new/burp.lintian-overrides
--- old/burp.lintian-overrides 2012-02-11 21:50:58.423862035 +0000
+++ new/burp.lintian-overrides 2012-02-11 21:53:31.695527885 +0000
@@ -1,4 +1,4 @@
-# lintian gives a waring becouse of file premisions witch is incorrect, file premission are for security reasons
+# the "wrong" file permissions are for security reasons
non-standard-file-perm etc/burp/burp-server.conf 0600 != 0644
non-standard-file-perm etc/burp/burp.conf 0600 != 0644
non-standard-dir-perm etc/burp/clientconfdir/ 0700 != 0755
diff -ru old/changelog new/changelog
--- old/changelog 2012-02-11 21:50:59.763861317 +0000
+++ new/changelog 2012-02-11 21:58:24.523861123 +0000
@@ -1,30 +1,30 @@
burp (1.3.0-2) unstable; urgency=low
* added forwarded header to patches
- * removed changelog entry's from patches
+ * removed changelog entries from patches
* removed debian/README (Closes: #659453)
- * removed blank mailto headers in cronjob and redirected out put to /var/log/burp-client (Closes: #659452)
- * added extra logrotate for /var/log/burp-client
- * Change /var/run to /run (Closes: #659464)
+ * removed blank MAILTO header in cronjob and redirected output to /var/log/burp-client (Closes: #659452)
+ * added extra logrotate for /var/log/burp-client
+ * changed /var/run to /run (Closes: #659464)
-- Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net> Sat, 11 Feb 2012 10:50:52 +0100
burp (1.3.0-1) unstable; urgency=low
- * changed files premisions in rule file
+ * changed file permissions in rules file
* removed empty todo and readme
- * Fixed an typo in the control file
- * override_dh_auto_configure restore is neasery to run
+ * fixed a typo in the control file
+ * override_dh_auto_configure restore is necessary to run
./configure without any options
- the configure files places the files in the correct
- directory voor a debian build
+ the configure files places the files in the correct
+ directory for a Debian build
* added watch file
* made patch to move PID file
- * Fixed lantian errors
- * Corrected the Copyright file
- * Initial release (Closes: #658152)
- * switches to 3.0 (quilt)
- * made lantian override file for file premissions
+ * fixed lintian errors
+ * corrected the Copyright file
+ * initial release (Closes: #658152)
+ * switched to 3.0 (quilt)
+ * made lintian override file for file permissions
* added man pages for bedup and burp_ca
diff -ru old/control new/control
--- old/control 2012-02-11 21:51:00.807861180 +0000
+++ new/control 2012-02-11 21:59:24.723527785 +0000
@@ -11,8 +11,7 @@
Package: burp
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}
-Description: backup and restore program
- Burp is a backup and restore program. It uses librsync in order to save
- the amount of space that is used by each backup. It also uses VSS (Volumer
- Shadow Copy Service) to make snapshots when backing up Windows computers.
-
+Description: simple network BackUp and Restore Program
+ Burp is a client/server backup system that saves space and network
+ usage by using librsync, and uses the Volume Snapshot Service when
+ backing up Windows clients.
# the "wrong" file permissions are for security reasons
non-standard-file-perm etc/burp/burp-server.conf 0600 != 0644
non-standard-file-perm etc/burp/burp.conf 0600 != 0644
non-standard-dir-perm etc/burp/clientconfdir/ 0700 != 0755
non-standard-file-perm etc/burp/ssl_cert-client.pem 0600 != 0644
non-standard-file-perm etc/burp/ssl_cert-server.pem 0600 != 0644
non-standard-dir-perm var/spool/burp/ 0700 != 0755
burp (1.3.0-2) unstable; urgency=low
* added forwarded header to patches
* removed changelog entries from patches
* removed debian/README (Closes: #659453)
* removed blank MAILTO header in cronjob and redirected output to /var/log/burp-client (Closes: #659452)
* added extra logrotate for /var/log/burp-client
* changed /var/run to /run (Closes: #659464)
-- Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net> Sat, 11 Feb 2012 10:50:52 +0100
burp (1.3.0-1) unstable; urgency=low
* changed file permissions in rules file
* removed empty todo and readme
* fixed a typo in the control file
* override_dh_auto_configure restore is necessary to run
./configure without any options
the configure files places the files in the correct
directory for a Debian build
* added watch file
* made patch to move PID file
* fixed lintian errors
* corrected the Copyright file
* initial release (Closes: #658152)
* switched to 3.0 (quilt)
* made lintian override file for file permissions
* added man pages for bedup and burp_ca
-- Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net> Fri, 10 Feb 2012 17:14:13 +0100
Source: burp
Section: utils
Priority: extra
Maintainer: Bastiaan Franciscus van den Dikkenberg <bas@dikkenberg.net>
Build-Depends: debhelper (>= 7.0.50~), autotools-dev, libncurses5-dev, libssl-dev, zlib1g-dev, librsync-dev, libacl1-dev, libattr1-dev
Standards-Version: 3.9.2
Homepage: http://burp.grke.net/
Vcs-Git: git://github.com/grke/burp.git
#Vcs-Browser: http://git.debian.org/?p=collab-maint/burp.git;a=summary
Package: burp
Architecture: any
Depends: ${shlibs:Depends}, ${misc:Depends}
Description: simple network BackUp and Restore Program
Burp is a client/server backup system that saves space and network
usage by using librsync, and uses the Volume Snapshot Service when
backing up Windows clients.
Reply to: