Bug#679300: RM: ruby-commandline -- ROM; Not compatible with current default Ruby version

2012-07-01 Thread Georgios M. Zarkadas
On Wed, Jun 27, 2012 at 05:46:23PM -0500, Gunnar Wolf wrote:
 Luca Falavigna dijo [Wed, Jun 27, 2012 at 11:41:21PM +0200]:
  tags 679300 + moreinfo
  thanks
  
  
  Not ready yet:
  # Broken Depends:
  dhelp: dhelp
  
 
 Hi,
 
 Grr, it seems the dhelp version I just uploaded contains my fix - But
 does *not* (and, sigh, I didn't check for it!) drop the dependency
 from debian/control.
 
 I'm cc:ing Georgios Zarkadas, the maintainer. Georgios, given I did
 the upload, do you mind if I upload a 0.6.21.1 (NMU) fixing this
 trivial issue? 
 
 Thanks,

Yes, proceed. 

George

PS: Sorry I didn't saw this earlier but a recent NetworkManager update changed 
my network interfaces numberings and messed with my fetchmailrc.



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#650441: ITA: dhelp -- online help system

2012-06-22 Thread Georgios M. Zarkadas
Hi all,

Switching to ruby 1.9.1 was left for next version (0.6.22), in order to
have 0.6.21 ready before the freeze (it was not a trivial change because
the test suite fails when using 1.9.1 and -cf. the changelog- I had
already a long list of things to be done).

When I find a sponsor and have the new version uploaded, I will start to
work on the interpreter transition.

Regards,
George Zarkadas



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#678055: dhelp: Porting dhelp to use OptionParser instead of CommandLine::Application

2012-06-21 Thread Georgios M. Zarkadas
Done, 
thanks for the patch! Already updated source tree in
http://anonscm.debian.org/gitweb/?p=collab-maint/dhelp.git;a=summary
and added a new version of the package at mentors.debian.net
(http://mentors.debian.net/debian/pool/main/d/dhelp/dhelp_0.6.21.dsc).

I only hope to manage find a sponsor, to close #677622, so that this
version makes it to Wheezy.

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#677622: RFS: dhelp/0.6.21 -- Debian online help [New version to also close #678055]

2012-06-21 Thread Georgios M. Zarkadas
Hi,

I have uploaded a new version of the dhelp package to also address bug
#678055, which showed up shortly after my initial RFS.

In case someone is reviewing the previous version of the package, the
changes are located in only one file: src/dhelp_parse.rb (and also
debian/changelog, of course).

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#677622: RFS: dhelp/0.6.21 -- Debian online help

2012-06-15 Thread Georgios M. Zarkadas
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package dhelp:

dget -x
http://mentors.debian.net/debian/pool/main/d/dhelp/dhelp_0.6.21.dsc

It builds these binary packages:

  dhelp - Debian online help

More information about dhelp can be obtained from
http://anonscm.debian.org/gitweb/?p=collab-maint/dhelp.git.

Changes since the last upload:

dhelp (0.6.21) unstable; urgency=low

  * New maintainer: Georgios M. Zarkadas g...@member.fsf.org (Closes: 
#650441). 
  * Support other web servers in addition to apache2 (Closes: #669041).
  * Support apache2 packaging transition for version 2.4 (Closes: #669758).
  * Support new ruby packaging policy transition for Wheezy.
  * Support new layout of man2html cgi scripts for Wheezy.
  * Keep supporting previous policies/layouts, either during build time or
during runtime, to aid backporting.
  * Man and info pages links are activated only if associated packages are
installed on the system.
  * Subsections now show in the sections list only if section is selected.
  * New color, styles and icons themes.
  * Package installation now does not fail if cache data cannot be generated
during install.
  * Fix some minor lintian warnings.
  * Bump Standards-Version to 3.9.3.

 -- Georgios M. Zarkadas g...@member.fsf.org  Tue, 12 June 2012 21:50:00 +0300

regards
George Zarkadas



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#665851: Bug#597050: GNU parallel, name conflict with moreutils

2012-05-25 Thread Georgios M. Zarkadas
Στις 24-05-2012, ημέρα Πεμ, και ώρα 21:11 +0100, ο/η Ian Jackson έγραψε:
 Joey Hess writes (Bug#665851: Bug#597050: GNU parallel, name conflict with 
 moreutils):
  Ian Jackson wrote:
   Ah, I'm sorry, I must have misunderstood your comments in #597050.  
   I took them for an objection, rather than a request for help.
  
  I sent a grand total of two sentances to that bug report.
  I will assume that I have not had any time to work on it did not
  cause that misunderstanding. This leaves:
 
 This situation still seems unresolved.
 
 What do other tech-ctte members think ?
 
 Ian.
 

Hi,

just FYI, GNU parallel is already in Wheezy, in a way that does not
conflict with moreutils (cf. the last messages in Bug#518696).

Regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#669041: Confirmed: dhelp: 404 Not Found

2012-04-18 Thread Georgios M. Zarkadas
Confirmed the bug using a minimal sid chroot where bozohttpd and dhelp were
installed with default configurations.  The fix is to insert a symlink with
name doc inside /var/www pointing to /usr/share/doc. That is, issue (as root)
the command:

ln -s /usr/share/doc /var/www/doc

The bug went un-noticed because the default up-to recently configuration of
apache included an Alias /doc /usr/share/doc directive. However this isn't
anymore true due to CVE-2012-0216 (cf. for example apache2_2.2.16-6+squeeze7
changelog); so this bug will also show up with a recent apache installation.

I will review the situation and prepare a fix asap.

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#664138: openclipart: Version 2.0 is not an upgrade of 0.18+dfsg but another package

2012-03-15 Thread Georgios M. Zarkadas
Package: openclipart
Version: 0.18+dfsg-10
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


The clipart contained in version 2.0 is an almost totally different set
of images. From the initial 8121 clips of ver. 0.18+dfsg, only about 582
survive (the about because some of the duplicates are packaging files).
See table.

Table: Total files and duplicates within each and accross both versions
- --- 
separate svg files in ver. 0.18+dfsg:  8121
separate svg files in ver. 2.0  : 13397

total duplicates in both versions   :  1346
duplicates inside ver. 0.18+dfsg:   548
duplicates inside ver. 2.0  :   216
duplicates accross both versions:   582
- ---

Thus replacing 0.18+dfsg with 2.0 will result in a totally different directory
structure, rendering any existing symlinks in the user's system (for example
folder images in nautilus) invalid, as well as loosing about:

8121 - 582 = 7639 clips.

Another problem (albeit a difficult one to solve, because it requires to find
a way to categorize all images) is that upstream replaced the category-based
organisation of files with an author-based one.

Both of the above, suggest that both packages should be co-installable (a
possible solution being for openclipart-2.0 to be renamed openclipart2)
or, if that is not desired, the old version's clipart should be merged with
the new version's one (minus the duplicates).

regards
George Zarkadas

PS-1: If I can come up with an elegant solution to categorize version's 2.0
clipart, I 'll file a whishlist bug for that.

PS-2: Setup used to calculate table figures.

* unpacked with dpkg-source -x both packages.
* from the directory above the packages' directories, issued these commands:

find openclipart-0.18+dfsg/ -path '*/clipart/*.svg' | wc --lines
find openclipart-2.0/ -path '*/clipart/*.svg' | wc --lines
fdupes -r openclipart-[02].[10]* | tee dups
dups awk '
BEGIN{
  RS=; FS=\n; a=0; b=0; c=0
}
{
  cnt[0]=cnt[1]=cnt[2]=0
  for (i=1;i=NF;++i){
if (match($i,openclipart-0.18)) cnt[1]=1
if (match($i,openclipart-2.0)) cnt[2]=1
  }
  cnt[0]=cnt[1]*cnt[2]
  if (cnt[0]) ++c
  else if (cnt[1]) ++a
  else if (cnt[2]) ++b
}
END{
  print v0.18 :  a
  print v2.0  :  b
  print both  :  c 
  print total :  a+b+c 
}'


- -- System Information:
Debian Release: 6.0.4
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages openclipart depends on:
ii  openclipart-openoffice.org  0.18+dfsg-10 clip art for OpenOffice.org galler
ii  openclipart-png 0.18+dfsg-10 clip art in PNG format
ii  openclipart-svg 0.18+dfsg-10 clip art in SVG format

openclipart recommends no packages.

openclipart suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJPYkmfAAoJEJWXIVmJ5BwWt+8IALsrN5JfemPy8OBTO8QMl7oM
8w/DLOuVlUPeQs6fT5iECeid7nu44ijCtX37OCIxc7hJgz7g36mzxeaGGa8nHUmk
0LnDwjRR0TX7awRfxV/PqVuL1hgPjrg9ARe2jBiSXaRRAqPjPgz8/BbkPD9oQktN
iha9UOr+k+n5ZMfumoGx4YGH53hRHg1qUe+GbzUl+CHa9Yz19/P7znuMZ8ZUilaz
hcGFTFzcxbITHiz0zJ/9+aWlsLwHPpKpbEcIrrPC5EExaKUBI4bnq1GaZWa9/l86
gcks/D3wXvf3oN7/XJY4uReKntmQouxSzfqAjlLeK+sIE5I51qDeqOKgrOnOVbw=
=OoTE
-END PGP SIGNATURE-



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#518696: [Fwd: Re: Progress?]

2012-03-04 Thread Georgios M. Zarkadas
Sorry for the previous message, it left my console incomplete;
continuing with the intended reply:

 Προωθημένο Μήνυμα 
Από: Georgios M. Zarkadas georgios.zarka...@gmail.com
Προς: Rogério Brito rbr...@ime.usp.br
Θέμα: Re: Progress?
Ημ/νία: Mon, 05 Mar 2012 01:30:00 +0200

Στις 04-03-2012, ημέρα Κυρ, και ώρα 18:44 -0300, ο/η Rogério Brito
έγραψε:
 Hi, Ondřej.
 
 On Sun, Mar 4, 2012 at 18:12, Ondřej Surý ond...@sury.org wrote:
  any progress on uploading to unstable?  I can help/sponsor/comaintain.
  If you don't respond, I'll just pick whatever is in collab-maint,
  check it (is the conflict with moreutils solved) and upload to
  unstable?
 
 Just for the record, I have updated the packaging and pushed it to 
 collab-maint:
 
 http://anonscm.debian.org/gitweb/?p=collab-maint/parallel.git
 
 Please let me know if there is anything else that I should do.
 
 Regards,
 

You must do either of following:

[1] Declare the package's conflict with moreutils in the control
file, because both packages share a binary in the path (/usr/bin/parallel).
That, is the debian/control file must contain a line:

Conflicts: moreutils

Also, it may be needed (someone with more experience in policy should tell)
that moreutils also declares that conflict.

This is for the case that the default (--gnu) configuration is chosen for
the package.

[2] Use the --tollef personality as default. Then there is no need for
a Conflicts, nor a need to modify moreutils. You just have to use dpkg-divert
in the install scripts to divert the moreutils' parallel with your package's
version.

Regarding ikiwiki-hosting-web, I believe that its only dependency from
moreutils is the use of chronic in its cron job. In any case a conflict with
that package is not necessary; the conflict with moreutils will be enough.

Also, for the case that --tollef / dpkg-divert is chosen, a later change by
the site admin of parallel's personality to --gnu will not interfere with the
ikiwiki-hosting-web package.

Just, FYI, the mentors thread, where the previous package's attempt was
discussed is: 
http://lists.debian.org/debian-mentors/2011/04/msg00203.html

I have since that time, taken other paths inside Debian and my interest
in packaging parallel has diminished; however if you feel you need help
with the dpkg-divert thing (assuming you choose the option [2]) I can
devote a small fraction of time to that.

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#661072: highlight: HLCmdLineApp::getFileSuffix should search for dots only in basename of the argument

2012-02-23 Thread Georgios M. Zarkadas
Package: highlight
Version: 2.16-1
Severity: normal
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

getFileSuffix returns all the part of the path passed in as argument
after the last dot, without considering whether a path-separator is
inside it. This leads to incorrect suffix recognition when the file
does not have a suffix, but the path leading to it has a dot, such as
for example:

../some-file (returns ./some-file)
/path-with.inside/another-file   (returns .inside/another-file)

The attached patch [1] fixes that on version's 3.7-2 git repository.

The attached patch [2] resolves this issue on version 2.16-1 (tested
building, installing and running a modified local version). 

regards
George Zarkadas

References:
[1] Version 3.7-2 patch:
0001-Search-for-file-suffix-only-in-filename-s-basename.patch
[2] version 2-16-1 patch:
0001-Fix-false-suffix-recognition-of-filenames-with-a-dot.patch 

- -- System Information:
Debian Release: 6.0.4
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages highlight depends on:
ii  highlight-common 2.16-1  source code to formatted text conv
ii  libc62.11.3-3Embedded GNU C Library: Shared lib
ii  libgcc1  1:4.4.5-8   GCC support library
ii  libstdc++6   4.6.2-12GNU Standard C++ Library v3

highlight recommends no packages.

highlight suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJPRtHbAAoJEJWXIVmJ5BwWmrUH/jvZPPxBYaCsRhQLi+Cf/p/G
MKalbK74WzPdey1poMWpMKknQViZJDZui0L+91poq+VX0RdyIM+mNnTBP1EeOEUD
smYSae0pcLMvkn5Fq8zLTCAIAidRl/87gK9p14hSEIVcNPDD9M5mvVPi/idm/x5N
SS/XGG/iEVs82oSFIQQp7CUj9w4yMnqAJjiFbkvqxxUw48A6C7COLyCw5d1Qb567
S2O7b1Kj9TLvmq4FwlQlR5paeKV/t/oAYfQ8DDSvqLngtLWUmG9uHwQfWbiiO+/r
K+0PtuWSYRWEqnkY/WNzN2BvHA5TFd0ePH8Nqqd/CBHUH7ymmSh6vjDGOHDl8M4=
=RYfu
-END PGP SIGNATURE-
From 18733cfaa05bbd9956ced1333c20f95f0d2cff0d Mon Sep 17 00:00:00 2001
From: Georgios M. Zarkadas g...@member.fsf.org
Date: Fri, 24 Feb 2012 01:22:31 +0200
Subject: [PATCH] Search for file suffix only in filename's basename.

Make the HLCmdLineApp::getFileSuffix member function to return a non-empty
file suffix only if the dot is found within the basename of the path that
is supplied as an argument.

This stops false positives when filenames without a suffix, but with dots
inside the path name (such as ../file or have.between/file) are given.
---
 src/cli/main.cpp |7 ++-
 1 files changed, 6 insertions(+), 1 deletions(-)

diff --git a/src/cli/main.cpp b/src/cli/main.cpp
index 10751f8..cfa9c1f 100644
--- a/src/cli/main.cpp
+++ b/src/cli/main.cpp
@@ -227,7 +227,12 @@ void HLCmdLineApp::printConfigInfo ( const string 
configFile )
 string HLCmdLineApp::getFileSuffix ( const string fileName )
 {
 size_t ptPos=fileName.rfind ( . );
-return ( ptPos == string::npos ) ?  : fileName.substr ( ptPos+1, 
fileName.length() );
+size_t psPos = fileName.rfind ( Platform::pathSeparator );
+
+if ( ptPos  psPos  ptPos != string::npos )
+return fileName.substr ( ptPos+1, fileName.length() );
+else
+return ;
 }
 
 bool HLCmdLineApp::loadFileTypeConfig ( const string name, StringMap* extMap, 
StringMap* shebangMap )
-- 
1.7.9

From a38680a7dd2a3ba97369df9ad988c028b35c07fe Mon Sep 17 00:00:00 2001
From: Georgios M. Zarkadas g...@member.fsf.org
Date: Thu, 23 Feb 2012 01:42:48 +0200
Subject: [PATCH 1/2] Fix false suffix recognition of filenames with a dot
 outside the basename.

---
 debian/patches/fix-get-file-suffix |   28 
 debian/patches/series  |1 +
 2 files changed, 29 insertions(+), 0 deletions(-)
 create mode 100644 debian/patches/fix-get-file-suffix

diff --git a/debian/patches/fix-get-file-suffix 
b/debian/patches/fix-get-file-suffix
new file mode 100644
index 000..9bedd3d
--- /dev/null
+++ b/debian/patches/fix-get-file-suffix
@@ -0,0 +1,28 @@
+Description: Search for file suffix only in filename's basename.
+ Make the HLCmdLineApp::getFileSuffix member function to return a non-empty
+ file suffix only if the dot is found within the basename of the path that
+ is supplied as an argument.
+ .
+ This stops false positives when filenames without a suffix, but with dots
+ inside the path name (such as ../file or have.between/file) are given.
+Author: Georgios M. Zarkadas g...@member.fsf.org
+Last-Update: 2012-02-23
+
+--- a/src/cli/main.cpp
 b/src/cli/main.cpp
+@@ -205,8 +205,13 @@
+ 
+ string HLCmdLineApp::getFileSuffix ( const string fileName )
+ {
+-  size_t ptPos

Bug#567553: backup-manager: config file problems after upgrade

2012-02-13 Thread Georgios M. Zarkadas
From the OP's message:
 ...
 This is not what was set in /etc/backup-manager.conf at that time.
 For me it looks like the config file is not read, maybe because of
 the warning, and debconf uses some kind of defaults for its report. 
 I had changed the config file by hand, but AFAIK it was correct ...

The config file is read. The warning is unrelated to this issue. Changes
by hand do not reflect to debconf data and this is by design: one has to
call debconf to update information and this currently is done by using:

dpkg-reconfigure package

So this part of the report is not a bug of backup-manager. It is a common
to all packages behaviour when a configuration file is edited by hand.

From the maintainer's message:
 ... In version 0.7.9 a new variable
 BM_LOGGER_LEVEL was introduced, and if that variable is not defined in
 backup-manager.conf you receive that incomprehensible warning.  It seems
 that is because logging and warnings are disabled until the sanitizing
 script (/usr/share/backup-manager/sanitize.sh) sets BM_LOGGER_LEVEL to
 the default value of warning.

Correct and indeed this is a bug, in the sense that all warnings/errors 
should be reported from the beginning, not just the final summary message
about the total number of errors (the reporting is per-se a feature, not
a bug).

This is easily fixed by cut+paste of the BM_LOGGER block in lib/sanitize.sh
from the end where currently is to the top of the checking code. I will
prepare a patch for the next-to-be-uploaded version asap, so that we can
close this bug then.

Cheers,
George



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#659325: highlight: Shebangs match the entire first line and not the start of it

2012-02-10 Thread Georgios M. Zarkadas
Package: highlight
Version: 2.16-1
Severity: normal
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

As is, the shebang recognition code matches the shebang against the entire
first line of the file. 

This has the undesired effect that shebangs which contain switches after the
executable (like for example: #!/usr/bin/awk -f) do not get recognised. Also
executable variants (like for example python2.6, python3, etc.) do not get
recognised, at least by the default-shipped regexes.

For some languages (like awk, make) the presence of switches are mandatory
in a shebang. For others are common. And in all cases, this is a departure
from the commonly expected behaviour when a ^some text regex is seen,
which is to match at the start of the line.

The attached patch makes shebangs match at the start of the string. 

Because the package format is quilt, I attach the contents of debian/patches
instead of a diff (file: patch1.tar.bz2); this makes it IMHO easier to pass
the patch upstream. I can also supply a diff if requested.

The report also applies to the 3.7 series of the package (same code there). 

I thus attach the patch modified to apply to the collab-maint git repository
(file: 0001-Make-shebang-regexes-match-at-the-start-of-the-line.patch.tar.bz2)
with git-apply. While the later is at 3.5, I have checked the 3.7 tarball and
the related code is the same; thus the git patch should apply without changes.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.4
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages highlight depends on:
ii  highlight-common 2.16-1  source code to formatted text conv
ii  libc62.11.3-2Embedded GNU C Library: Shared lib
ii  libgcc1  1:4.4.5-8   GCC support library
ii  libstdc++6   4.4.5-8 The GNU Standard C++ Library v3

highlight recommends no packages.

highlight suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJPNNNqAAoJEJWXIVmJ5BwWdxkH/2vmoVOPp5/6tW/wur7P4E+t
4VKPajU5LfMoBb0j39QJWoUclMTFnUHS6mVdalV77yFj9MFSWViijaH1Fon128Du
B6bOy4VABb7kdiSGgx8TLvGq1M9AuC95pR+kjod5GGeBK285pOJJm8CNCZaPOdPb
41AwB5RB/KXSCVt5Sys8fd2+VrC/jj2UVHHBJDnb1jmcG8VJO/JtXgpWSuMsRl2Y
lvi2dIwLXTCTnxHV0hpU2x7e+uiHbiV6iV0oUA9UX7368BTtIuZipPjMKNqhkIsq
fBqqDx+StrGBg8K+7Gs2JGtPoEkAfkM+h3eBEmhB2x06BgVSvXOBYFtKiaB6Tb4=
=Vt6y
-END PGP SIGNATURE-


patch1.tar.bz2
Description: BZip2 compressed data


0001-Make-shebang-regexes-match-at-the-start-of-the-line.patch.tar.bz2
Description: BZip2 compressed data


Bug#659384: highlight: Shebang regexes to cover more use cases

2012-02-10 Thread Georgios M. Zarkadas
Package: highlight
Version: 2.16-1
Severity: wishlist
Tags: upstream

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Attached is a patch (file: patch2.tar.bz2) with updated and new shebang
regexes to account for the following use cases:

 * Whitespace between the starting #! and the executable's path.
 * Usage of the #!/usr/bin/env executable form of shebang used
   by some scripts.
 * Recognition of ruby, php and make executable scripts.
 * Recognition of the dash shell and the mawk awk variant.

Many of the above use cases are not corner-case; a survey on my /usr/bin
(which is certainly biased, since no attempt was made to acquire a random
set of packages before running it, but also indicative of the orders of
magnitude involved) showed the following results:

Shebang Type Count  Percentage
- --
all (binaries excluded)   1253
with spaces25821 %
with /usr/bin/env   71 6 %
with options after 32226 %
- --

Since the package uses the quilt format, I attach directly the files in 
debian/patches, to make it also easier to get applied upstream. A diff 
can also be supplied on request.

I also attach a patch for the collab-maint git version (file: 
0002-Change-shebang-regexes-to-account-for-spaces-and-the.patch) which
is currently 3.5 as I see, but have also verified compatibility with the
3.7 version.

A side note: In my tests with the 2.16 version the regexes had not required
the preceeding backslash; I have thus dropped it to make the regexes more
readable.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.4
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages highlight depends on:
ii  highlight-common 2.16-1  source code to formatted text conv
ii  libc62.11.3-2Embedded GNU C Library: Shared lib
ii  libgcc1  1:4.4.5-8   GCC support library
ii  libstdc++6   4.4.5-8 The GNU Standard C++ Library v3

highlight recommends no packages.

highlight suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJPNUrgAAoJEJWXIVmJ5BwWI/QIAL5hm/mSFYrhiYbElk4M6KvG
xpGCKELkrfZTr6rErzdhuPsZkacr0jvPClC3FYhpo+hOcekCe0rIa51+v4pjWVpW
2A89Gi3WQ4bHb8GogZpuFPsuPIjGhwTZJBIvaTz5Zw+C2dqzsHAItZEcXpckauZ0
rxuK3Kqoip3tXf2KMAnYZmVhE0e66jfIqpRrw5M6vECXxcw0aeREO6sSjkcZtOLS
94bJfDtGHeBdkVy3/YvsPYNsJgqYzYIfOx88M1E8/Yi8flK+uMwlafVdQjbr2Q/T
eK/8PKOyuAgA1N5NZn12zLOC0dCxuyhBOP76Bevgdb3im2KOMHWMRXsdV9d1Bp4=
=Wenc
-END PGP SIGNATURE-


patch2.tar.bz2
Description: BZip2 compressed data


0002-Change-shebang-regexes-to-account-for-spaces-and-the.patch.tar.bz2
Description: BZip2 compressed data


Bug#491199: backup-manager: Unspecified dependicies for uploading to Amazon S3

2012-02-08 Thread Georgios M. Zarkadas
A few remarks, after looking the source:

-- libfile-slurp-perl is not a dependency anymore from version 0.7.9
(cf. the diff between commits 9a00cd (version 0.7.9) and 91830b (version
0.7.8) at upstream's cvs: https://github.com/sukria/Backup-Manager ,
branch 'releases').

-- libnet-amazon-s3-perl is indeed a dependency. However upstream code
does test for module's existence and fails gracefully if it is not
present. By gracefully I mean that it does signal an error and logs to
syslog (through the 'print_error' function) if the module is not
present. I believe this was done at 0.7.9 version, also. 

Thus libnet-amazon-s3-perl is a soft dependency (deserving a Suggests)
and all issues reported by the OP are now fixed by upstream.

Therefore, I propose to update the control file, removing the
libfile-slurp-perl from the Suggests section (libnet-amazon-s3-perl is
already there) and close the bug.

Cheers,
George


signature.asc
Description: This is a digitally signed message part


Bug#638919: backup-manager: Please use a single md5 sums file

2012-01-24 Thread Georgios M. Zarkadas
Στις 23-01-2012, ημέρα Δευ, και ώρα 19:25 +0100, ο/η Sven Joachim
έγραψε:
 On 2012-01-22 17:44 +0100, Georgios M. Zarkadas wrote:
  Στις 21-01-2012, ημέρα Σαβ, και ώρα 14:00 +0100, ο/η Sven Joachim
  έγραψε:
  On 2012-01-16 17:58 +0100, Sven Joachim wrote:
  
   It seems that we need a migration path to the single md5sums file, ...
  ...
 ...
 Another place where the md5sums files are used is the burning facility
 if BM_BURNING_CHKMD5 is set to true.  Have you investigated that yet?

Burning was not covered by the initial patch. In that respect the md5
sums test are now broken as a whole due to the patch. I have already
made an amendment to fix this, but it will need testing, so it will
delay a new release of the package.

  ...
 
 In the light of this, how about disabling the single md5sum feature in
 the next upload and getting it into better shape upstream first?

Considering that -although accepted upstream- the patch has not yet
appeared in an upstream release, as well as the issues mentioned here,
this is the best way to go for the moment.

Cheers,
George


signature.asc
Description: This is a digitally signed message part


Bug#638919: backup-manager: Please use a single md5 sums file

2012-01-22 Thread Georgios M. Zarkadas
Στις 21-01-2012, ημέρα Σαβ, και ώρα 14:00 +0100, ο/η Sven Joachim
έγραψε:
 On 2012-01-16 17:58 +0100, Sven Joachim wrote:
 
  It seems that we need a migration path to the single md5sums file, since
  (with backup-manager from git master) cron spams me with mails
  containing messages like these:
 ...
  After BM_ARCHIVE_TTL days this will hopefully stop, but it is still not
  acceptable to annoy everyone with those messages.
 
 There is another problem that applies to upstream as well: md5sum hashes
 for old backups are not removed from $MD5FILE when the archives
 themselves are deleted

Regarding the first, the possible solutions are:

*   Lower the severity of this condition to 'info' instead of 'warning'.
Needs to patch upstream, but it is just a one-line patch.

*   Launch a background process during the package's postinst script
to calculate the md5 hash of existing archives and store them to
the md5 sums file. This can be tricky; we don't know the size of 
archives beforehand, nor can we warranty that the process will not
get interrupted.

*   Modify just the cron file to grep out that message. That means to
add a '| grep -v Unable to find the md5 hash' to the template
and also to dynamically examine and modify any existing cronfile
during the postinst.

I am inclined to do the first of the above. The message is only issued
by the 'purge_duplicate_archives' function, thus the only harm that a
missing md5 hash can make is to miss a possible opportunity to free some
disk space before the BM_ARCHIVE_TTL period expires. 'Info' seems to me
a more appropriate severity level for such minor issues. 

Regarding the second issue, I believe the best way to solve it is to
ship a new monthly cronjob that uses the largest of BM_ARCHIVE_TTL and
BM_UPLOAD_*TTL settings, then removes all lines with older date from the
md5 sums file. It is premature to try to patch the upstream code,
because IMO the whole metadata subject needs a mild redesign.

Cheers,
George


signature.asc
Description: This is a digitally signed message part


Bug#615900: A series of patches to address this issue

2011-09-22 Thread Georgios M. Zarkadas
tags 615900 + patch
thanks

Hi,

I attach a set of three patches that fix this bug, to be applied in the
order they are numbered (...once_NUM-...).
[Please read notes at the end of this message on how to apply]

The core idea behind the patch-set is to defer the indexing until the
installation procedure is finished, and then perform it just once in the
background (by default incrementally, unless the index is missing, in
which case the indexer switches to full mode).

This of course required quite a number of changes that are briefly
presented below:

* Changes to the registering process workflow: the registered documents
are appended to a file, which will be read later by the indexer, instead
of being fed immediately to the indexer.

* The above required a number of changes to the underlying Ruby classes
(DHelpDocumentPool and Indexer), as well as to the dhelp_parse
front-end. 
  A test script was also touched, to comply with the changed code.

* Insertion of a Dpkg::Post-Invoke trigger to do the indexing at the end
of each installation run. 
  This also required using ucf in the maintainer scripts to handle its
creation/removal (in order to avoid errors interrupting the install
process during purge of the package).

* Insertion of some shell machinery as a wrapper to the indexing
commands, namely: 
  * an 'index-deferred' script that calls the real indexing commands,
  * use of nohup to ensure swish++ will keep running even if the
controlling terminal disappears,
  * logging nohup-ed activities' output to /var/lib/dhelp/tmp ,
  * a monthly cron job to cleanup log files inside that directory.

* Changes to build and packaging files (Makefile, maintainer scripts,
etc.)

As a side effect, dhelp_parse -r now rebuilds the pool, without starting
the indexer, and creates a pending list with all documents. Also the
Indexer class will switch to full mode from incremental if the index is
not found. Thus the sequence 'dhelp_parse -r ; dhelp_parse -i' can
recreate a missing index.

The patches have been tested regarding the following aspects:

* Test suite: all tests pass.

* Installation: install/upgrade/downgrade/remove/purge procedures and
combinations of the above, with versions 0.6.19 and the local ones from
the application of the patches.

* Registration of doc-base documents: addition/removal of various *-doc
packages after installation.

## NOTES: ##

* Application of the patches leaves two .orig files due to succeeding
with fuzz. This is OK; patches were crafted so that they can be
independently applied with regard to patches sent to other bug reports
(#561883 mainly, which also touches those two files).
Those files (Makefile.orig , debian/dirs.orig) should be manually
removed.

* The executable permissions of index-deferred and debian/cron.monthly
should be manually added after the patch.

* Patch number 3 incorporates the patch sent in #642273 bug report. This
was made for simplicity, to avoid sending and explaining pairs of
separate patches. If #642273 is not accepted, I will send a modified
version of the number 3 patch.

regards
George Zarkadas

--- a/dhelp_parse.rb
+++ b/dhelp_parse.rb
@@ -61,9 +61,9 @@
 version   0.2.0
 authorEsteban Manchado Velázquez
 copyright Copyright (c) 2005-2007, Esteban Manchado Velázquez
-synopsis  [-v] [-h] -a doc-base_file1 d-b_f2 ... | -d doc-base_file1 d-b_f2 ... | -r
+synopsis  [-v] [-h] -a doc-base_file1 d-b_f2 ... | -d doc-base_file1 d-b_f2 ... | -i | -r
 short_description Debian online help system parser
-long_description  Dhelp parser to add/remove/reindex dhelp files
+long_description  Dhelp parser to add/remove/index-incrementally/fully-reindex dhelp files
 
 option :help
 option :names = %w(-a), :arity = [0,-1],
@@ -77,9 +77,12 @@
 option :names = %w(-v), :arity = [0,0],
:opt_found = lambda { @verbose = true },
:opt_description = verbose
+option :names = %w(-i), :arity = [0,0],
+   :opt_found = lambda { @action = :index },
+   :opt_description = perform deferred incremental indexing of pending registered docs
 option :names = %w(-r), :arity = [0,0],
:opt_found = lambda { @action = :reindex },
-   :opt_description = ignored, for compatibility
+   :opt_description = perform full re-indexing of all registered docs
 
 expected_args [0,0]
 
@@ -87,11 +90,52 @@
 @verbose = false
   end
 
-
   def packaged_configured?
 File.exists? '/var/lib/dhelp/configured'
   end
 
+  # Adds the documents supplied in command-line to the pool.
+  def add_documents(pool)
+@doc_base_files.each do |doc_base_file|
+  if File.readable?(doc_base_file)
+if @verbose
+  puts Parsing document #{doc_base_file}
+end
+doc_base_doc = Dhelp::DocBaseDocument.new(doc_base_file)
+if @verbose
+  puts Registering document #{doc_base_file}
+end
+

Bug#642316: debsums: Greek translation of the package

2011-09-21 Thread Georgios M. Zarkadas
Package: debsums
Version: 2.0.48+nmu3
Severity: wishlist
Tags: patch l10n

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


Attached is a Greek translation for debsums. I attach both the .po file
and a diff of it (for applying with patch -p1); pick what suits you best.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages debsums depends on:
ii  debconf [debconf-2.0]  1.5.36.1  Debian configuration management sy
ii  perl   5.10.1-17squeeze2 Larry Wall's Practical Extraction 
ii  ucf3.0025+nmu1   Update Configuration File: preserv

debsums recommends no packages.

debsums suggests no packages.

- -- debconf information:
  debsums/apt-autogen: true

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOefBEAAoJEJWXIVmJ5BwWBNsH/1IBxOpEcoqrhS8p+AOnwMbf
opv2NooXDXir9q9Lj0UhafVaZiUNQonkLlZkhOZ+X43PC/rdz+DCxLulnVGecpaK
X1GUBlxV55yFwbl9N/RvYfxysM21FTjqEEEbAaz7B0jlh679e1+5E3Ac2wpGr1Z5
L3r5/GRrB//T6coORXjrsY2XJlP/lGonTejCR7cqxksAkCAM4NaByXxysd9YFVU4
wJ+96F8/nimJFJrPUZgD1D/mBfqagxho4WtUxNh/zEMWQ3FJKQ6lmc5OIn3Q9IFi
FFI+5lXuzSN3KCDPCZGkPlEs/3e6+ka3cbz/0KLsc478cKbu7rKy2W1xPKUyZRM=
=cAdy
-END PGP SIGNATURE-
# Greek translation of the debsums package.
# Copyright (C) 2011 THE PACKAGE'S COPYRIGHT HOLDER(S)
# This file is distributed under the same license as the debsums package.
# Georgios M. Zarkadas g...@member.fsf.org, 2011.
#
msgid 
msgstr 
Project-Id-Version: debsums 2.0.36, 2.0.48, 2.0.49\n
Report-Msgid-Bugs-To: debs...@packages.debian.org\n
POT-Creation-Date: 2009-06-26 16:22+1200\n
PO-Revision-Date: 2011-09-21 16:47+0300\n
Last-Translator: Georgios M. Zarkadas g...@member.fsf.org\n
Language-Team: el debian-l10n-gr...@lists.debian.org\n
MIME-Version: 1.0\n
Content-Type: text/plain; charset=UTF-8\n
Content-Transfer-Encoding: 8bit\n
Plural-Forms: \n

#. Type: boolean
#. Description
#: ../templates:1001
msgid Should debsums files be generated automatically by apt-get?
msgstr Να δημιουργούνται αυτόματα τα αρχεία debsums από το apt-get;

#. Type: boolean
#. Description
#: ../templates:1001
msgid 
Not all packages contain debsums information. However, apt can be configured 
to generate debsums files for installed packages automatically. This may be 
useful for checking system integrity later, but it should not be relied upon 
as a security measure.
msgstr 
Δεν περιέχουν όλα τα πακέτα πληροφορίες debsums. Ωστόσο το apt μπορεί να 
ρυθμιστεί ώστε να δημιουργεί αυτόματα αρχεία debsums για τα εγκατεστημένα 
πακέτα. Αυτό μπορεί να είναι χρήσιμο για τον έλεγχο της ακεραιότητας του 
συστήματος αργότερα, αλλά δεν πρέπει να θεωρηθεί ως ένα αξιόπιστο μέτρο 
ασφαλείας.
--- /dev/null   2011-09-18 07:17:39.162064342 +0300
+++ b/debian/po/el.po   2011-09-21 17:01:53.662075474 +0300
@@ -0,0 +1,38 @@
+# Greek translation of the debsums package.
+# Copyright (C) 2011 THE PACKAGE'S COPYRIGHT HOLDER(S)
+# This file is distributed under the same license as the debsums package.
+# Georgios M. Zarkadas g...@member.fsf.org, 2011.
+#
+msgid 
+msgstr 
+Project-Id-Version: debsums 2.0.36, 2.0.48, 2.0.49\n
+Report-Msgid-Bugs-To: debs...@packages.debian.org\n
+POT-Creation-Date: 2009-06-26 16:22+1200\n
+PO-Revision-Date: 2011-09-21 16:47+0300\n
+Last-Translator: Georgios M. Zarkadas g...@member.fsf.org\n
+Language-Team: el debian-l10n-gr...@lists.debian.org\n
+MIME-Version: 1.0\n
+Content-Type: text/plain; charset=UTF-8\n
+Content-Transfer-Encoding: 8bit\n
+Plural-Forms: \n
+
+#. Type: boolean
+#. Description
+#: ../templates:1001
+msgid Should debsums files be generated automatically by apt-get?
+msgstr Να δημιουργούνται αυτόματα τα αρχεία debsums από το apt-get;
+
+#. Type: boolean
+#. Description
+#: ../templates:1001
+msgid 
+Not all packages contain debsums information. However, apt can be configured 
+to generate debsums files for installed packages automatically. This may be 
+useful for checking system integrity later, but it should not be relied upon 
+as a security measure.
+msgstr 
+Δεν περιέχουν όλα τα πακέτα πληροφορίες debsums. Ωστόσο το apt μπορεί να 
+ρυθμιστεί ώστε να δημιουργεί αυτόματα αρχεία debsums για τα εγκατεστημένα 
+πακέτα. Αυτό μπορεί να είναι χρήσιμο για τον έλεγχο της ακεραιότητας του 
+συστήματος αργότερα, αλλά δεν πρέπει να θεωρηθεί ως ένα αξιόπιστο μέτρο 
+ασφαλείας.


Bug#642273: dhelp: The postrm script removes unrelated directory

2011-09-20 Thread Georgios M. Zarkadas
Package: dhelp
Version: 0.6.19
Severity: normal
Tags: patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

The postrm script removes directory /etc/dhelp. However this directory is not
created by the package during install since lenny (the last version I could
find which did, is 0.5.24 in etch). 

Thus /etc/dhelp is from quite a long time ago a free path. It is thus 
conceivable that an administrator could create (as part of a custom config or
for whatever reason) that directory inside /etc and populate it with data. 
Upon removal of dhelp that directory and all data inside it would unknowingly
be deleted without question. How small this possibility is, it is nevertheless
a bad thing for a package to do, IMHO.

The attached patch solves this issue.

regards
George Zarkadas 

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dhelp depends on:
ii  doc-base 0.9.5   utilities to manage online documen
ii  libcommandline-ruby1 0.7.10-10   Ruby library to write command-line
ii  libdata-page-perl2.02-1  Help when paging through sets of r
ii  libdb-ruby1.80.6.5-6 Interface to Berkeley DB for Ruby 
ii  libgettext-ruby1.8   2.1.0-2.1   Gettext for ruby1.8
ii  libhtml-parser-perl  3.66-1  collection of modules that parse H
ii  liblocale-gettext-pe 1.05-6  Using libc functions for internati
ii  libtemplate-perl 2.22-0.1template processing system written
ii  liburi-perl  1.54-2  module to manipulate and access UR
ii  perl-modules 5.10.1-17squeeze2   Core Perl modules
ii  poppler-utils0.12.4-1.2  PDF utilitites (based on libpopple
ii  pstotext 1.9-5   Extract text from PostScript and P
ii  ruby1.8  1.8.7.302-2squeeze1 Interpreter of object-oriented scr
ii  swish++  6.1.5-2 Simple Document Indexing System fo

Versions of packages dhelp recommends:
ii  epiphany-browser [www-browser 2.30.6-1   Intuitive GNOME web browser
ii  iceweasel [www-browser]   3.5.16-9   Web browser based on Firefox
ii  w3m [www-browser] 0.5.2-9WWW browsable pager with excellent

Versions of packages dhelp suggests:
ii  apache2-mpm-prefork [h 2.2.16-6+squeeze3 Apache HTTP Server - traditional n
ii  catdvi 0.14-11+b1DVI to plain text translator
ii  html2text  1.3.2a-15 advanced HTML to text converter
ii  info2www   1.2.2.9-24Read info files with a WWW browser
ii  man2html   1.6f+repack-1 browse man pages in your web brows
ii  w3m0.5.2-9   WWW browsable pager with excellent

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOeSZMAAoJEJWXIVmJ5BwW1N0H/jk9OZefmiw1IqTOewUK/bD1
hDFRpJKzC71rGrN0WR3zaJN9pN2/CaFcDjjWSsUUZ1bV1Hv4XZ+328GPTqtcykKq
f6LRX3PoV/sFbscB7bcgq+Ir1AYpV7JxC+MmhBfBtHJGytZnKFVzoKOECrwlcBCs
MyeL9fU4oJ8jeELT8nxNRyo4ebsv4aCIQNj0hdYLYtb6xEauPklHnpPWS+CBTBkQ
Uc9vr8s+CCk52iil1D8vjptzhjiygqaKDuJRGFTVNVLaGr4vdVSib8LLIJK05gLR
t11HbhSfDTQewdBHU3Cj4ehqK6v1Lh2wKitzG6OA8S9U/VfG84efhuLrnyfdqMc=
=wR3S
-END PGP SIGNATURE-
--- a/debian/postrm
+++ b/debian/postrm
@@ -17,9 +17,4 @@
 # Software Foundation, Inc., 59 Temple Place, Suite 330, Boston,
 # MA  02111-1307  USA.
 
-if [ $1 = purge ]; then
-# remove files generated by postinst
-   rm -rf /etc/dhelp
-fi
-
 #DEBHELPER#


Bug#554660: Patch to address the bug

2011-09-18 Thread Georgios M. Zarkadas
tags 554660 + patch
thanks

The attached patch builds the package successfully in a sid pbuilder
chroot with EXTRAPACKAGES=binutils-gold on an amd64 system. I attach
the build log also, FYI.

In my opinion this should be treated as a Debian-specific patch to make
the package build until upstream will incorporate the missing library
(libxklavier) into its build system.

regards
George Zarkadas
diff --git a/debian/control.in b/debian/control.in
index 6a6b4c6..7c823b8 100644
--- a/debian/control.in
+++ b/debian/control.in
@@ -18,7 +18,7 @@ Build-Depends: cdbs,
libgnomekbd-dev (= 2.24),
libxt-dev,
libxext-dev,
-   libxklavier-dev,
+   libx11-dev,
libpam0g-dev,
xmlto,
x11proto-xext-dev,
diff --git a/debian/rules b/debian/rules
index 9485038..5930bef 100755
--- a/debian/rules
+++ b/debian/rules
@@ -15,7 +15,7 @@ DEB_CONFIGURE_EXTRA_FLAGS += \
 			--without-libnotify \
 			--enable-docbook-docs
 
-LDFLAGS += -Wl,-z,defs -Wl,-O1 -Wl,--as-needed -lxklavier
+LDFLAGS += -Wl,-z,defs -Wl,-O1 -Wl,--as-needed
 
 binary-post-install/gnome-screensaver::
 	mv debian/gnome-screensaver/etc/xdg/autostart \


gnome-screensaver_2.30.0-3_amd64.build.tar.bz2
Description: application/bzip-compressed-tar


signature.asc
Description: This is a digitally signed message part


Bug#641917: dhelp: Greek localisation

2011-09-17 Thread Georgios M. Zarkadas
Package: dhelp
Version: 0.6.19+nmu1
Severity: wishlist
Tags: l10n patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


Attached is a patch to localise dhelp for the Greek language. 

The patch contains the relevant po file (el.po) and the addition of el_GR.utf8
locale to gettext.pl. 

Regarding the later, I took the liberty to also add the DE locale that was
missing and arrange all locales alphabetically in a form that makes future
additions easy to perform and inspect. If you don't like it, you can just
ommit this part of the patch and add 'el_GR.utf8' to the associated line 
in gettext.pl.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages dhelp depends on:
ii  bzip21.0.5-6 high-quality block-sorting file co
ii  doc-base 0.9.5   utilities to manage online documen
ii  libcommandline-ruby1 0.7.10-10   Ruby library to write command-line
ii  libdata-page-perl2.02-1  Help when paging through sets of r
ii  libdb-ruby1.80.6.5-6 Interface to Berkeley DB for Ruby 
ii  libgettext-ruby1.8   2.1.0-2.1   Gettext for ruby1.8
ii  libhtml-parser-perl  3.66-1  collection of modules that parse H
ii  liblocale-gettext-pe 1.05-6  Using libc functions for internati
ii  libtemplate-perl 2.22-0.1template processing system written
ii  liburi-perl  1.54-2  module to manipulate and access UR
ii  perl-modules 5.10.1-17squeeze2   Core Perl modules
ii  poppler-utils0.12.4-1.2  PDF utilitites (based on libpopple
ii  pstotext 1.9-5   Extract text from PostScript and P
ii  ruby1.8  1.8.7.302-2squeeze1 Interpreter of object-oriented scr
ii  swish++  6.1.5-2 Simple Document Indexing System fo

Versions of packages dhelp recommends:
ii  epiphany-browser [www-browser 2.30.6-1   Intuitive GNOME web browser
ii  iceweasel [www-browser]   3.5.16-9   Web browser based on Firefox
ii  w3m [www-browser] 0.5.2-9WWW browsable pager with excellent

Versions of packages dhelp suggests:
ii  apache2-mpm-prefork [h 2.2.16-6+squeeze2 Apache HTTP Server - traditional n
pn  catdvi none(no description available)
ii  html2text  1.3.2a-15 advanced HTML to text converter
ii  info2www   1.2.2.9-24Read info files with a WWW browser
ii  man2html   1.6f+repack-1 browse man pages in your web brows
ii  w3m0.5.2-9   WWW browsable pager with excellent

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOdKmQAAoJEJWXIVmJ5BwWpSIH/2DnyguS8n30JGUUstN2ZOAV
4rqWegTdl4nm08TNkENuvfCj/sbEHVQ9EFFs8jvYQD53p7lNV79py5jDvFoUq9Yv
v5lGIKQA39JCoXcdA9tpN2/zFh35J+7UPCTU4jXIWr33J/i2BA5iKQLhLBsaWDXC
3XlwniaLGWH/WEjvD+T7HnS+22cWZZqwSSf6M3k9/M5qR3ZdOwh1W2gaJMNvkMHT
NPJJFK7rQUExeEVxXU+wjNV/+DDARCcLbyR62Jlrej1k8X2RQJagJJM2OOTy+xWN
7et4u8/LiUA4GhHNdNggUNGmyShkPxvDpPxoJI3LpQEM/UBxGDwO4/xCaLKhxow=
=DsrI
-END PGP SIGNATURE-
--- a/gettext.pl
+++ b/gettext.pl
@@ -4,7 +4,15 @@
 my $gettext = Locale::gettext-domain_raw(dhelp);
 $gettext-codeset('UTF-8'); # Always UTF-8, specified in the HTML templates
 
-foreach my $locale (qw(id_ID.utf8 eu_ES.utf8 es_ES.utf8 ru_RU.utf8 
fr_FR.utf8)) {
+foreach my $locale (qw(
+de_DE.utf8
+el_GR.utf8
+es_ES.utf8
+eu_ES.utf8
+id_ID.utf8
+ru_RU.utf8
+fr_FR.utf8
+)) {
 setlocale(LC_MESSAGES, $locale);
 my $string = No search database found.\nPlease run /etc/cron.weekly/dhelp 
as superuser to create it.;
 print LOCALE $locale \n;
--- /dev/null
+++ b/po/el.po
@@ -0,0 +1,107 @@
+# Greek translation of dhelp messages.
+# Copyright (C) 2011 Georgios M. Zarkadas g...@member.fsf.org
+# This file is distributed under the same license as the dhelp package.
+# Georgios M. Zarkadas g...@member.fsf.org, 2011.
+#
+msgid 
+msgstr 
+Project-Id-Version: dhelp\n
+Report-Msgid-Bugs-To: \n
+POT-Creation-Date: 2008-02-19 23:02+0100\n
+PO-Revision-Date: 2011-09-08 23:43+0300\n
+Last-Translator: Georgios M. Zarkadas g...@member.fsf.org\n
+Language-Team: el debian-l10n-gr...@lists.debian.org\n
+MIME-Version: 1.0\n
+Content-Type: text/plain; charset=UTF-8\n
+Content-Transfer-Encoding: 8bit\n
+Plural-Forms: \n
+
+#: fetcher_error.rhtml:14 index.rhtml:14 section.rhtml:21 search_error.tmpl:15
+#: search_results.tmpl:15
+msgid Online Help
+msgstr Άμεση βοήθεια

Bug#561883: Patch to address the bug

2011-09-17 Thread Georgios M. Zarkadas
tags 561883 + patch
thanks

The attached patch solves the issue in all affected versions of dhelp
(up to 0.6-19).

What the patch does is to use shell scripts for decompression and
conversion. The scripts hide the -often quite large- output from the
tools and instead just output one line with the offending document's
filename. 

That way the cron job returns a small and easily comprehensible message
on errors. Anyone wishing to investigate the error can then manually
invoke the decompression/conversion tool upon the file.

The patch has been successfully tested with regard to package-building,
installation and correct operation on version 0.6-19 sources. 

regards
George Zarkadas
--- a/Makefile
+++ b/Makefile
@@ -44,6 +44,9 @@ install:
 	cp *.rhtml *.tmpl swish++.conf $(DESTDIR_)/share/dhelp/
 	mkdir -p $(DESTDIR)/etc
 	cp dhelp.conf-sample $(DESTDIR)/etc/dhelp.conf
+	mkdir -p $(DESTDIR_)/share/dhelp/scripts
+	cp scripts/* $(DESTDIR_)/share/dhelp/scripts/
+	chmod 755 $(DESTDIR_)/share/dhelp/scripts/*
 	# Translations
 	for i in po/*.po; do \
 	mkdir -p $(DESTDIR_)/share/locale/`basename $$i .po`/LC_MESSAGES/; \
--- a/debian/dirs
+++ b/debian/dirs
@@ -1,4 +1,5 @@
 usr/share/dhelp
+usr/share/dhelp/scripts
 var/lib/dhelp
 var/lib/dhelp/tmp
 etc/apache2/conf.d
--- /dev/null
+++ b/scripts/conv-pdftotext
@@ -0,0 +1,14 @@
+#!/bin/sh
+# Script optimised to run from within dhelp's swish++.conf.
+# That means:
+# * No error checking on parameters; edit swish++.conf the right way.
+# * Spit output only on errors (a line pointing to input file).
+#
+# $1 = Input file, $2 = Output file
+
+2/dev/null pdftotext ${1} ${2}
+EXITVAL=$?
+if [ ${EXITVAL} -ne 0 ]; then
+	echo Error converting file: ${1}
+fi
+exit ${EXITVAL}
--- /dev/null
+++ b/scripts/conv-pstotext
@@ -0,0 +1,14 @@
+#!/bin/sh
+# Script optimised to run from within dhelp's swish++.conf.
+# That means:
+# * No error checking on parameters; edit swish++.conf the right way.
+# * Spit output only on errors (a line pointing to input file).
+#
+# $1 = Input file, $2 = Output file
+
+2/dev/null pstotext -output ${2} ${1}
+EXITVAL=$?
+if [ ${EXITVAL} -ne 0 ]; then
+	echo Error converting file: ${1}
+fi
+exit ${EXITVAL}
--- /dev/null
+++ b/scripts/gz-extract
@@ -0,0 +1,14 @@
+#!/bin/sh
+# Script optimised to run from within dhelp's swish++.conf.
+# That means:
+# * No error checking on parameters; edit swish++.conf the right way.
+# * Spit output only on errors (a line pointing to input file).
+#
+# $1 = Input file, $2 = Output file
+
+2/dev/null gzip -dc ${1}  ${2}
+EXITVAL=$?
+if [ ${EXITVAL} -ne 0 ]; then
+	echo Error decompressing file: ${1}
+fi
+exit ${EXITVAL}
--- a/swish++.conf
+++ b/swish++.conf
@@ -1,8 +1,9 @@
 TempDirectory /var/lib/dhelp/tmp
 WordThreshold 5
-FilterFile *.gzgunzip -c %f  @/var/lib/dhelp/tmp/%B
-FilterFile *.pdf   pdftotext %f @/var/lib/dhelp/tmp/%B.txt
-FilterFile *.pspstotext  %f  @/var/lib/dhelp/tmp/%B.txt
+
+FilterFile *.gz   /usr/share/dhelp/scripts/gz-extract %f @/var/lib/dhelp/tmp/%B
+FilterFile *.pdf  /usr/share/dhelp/scripts/conv-pdftotext %f @/var/lib/dhelp/tmp/%B.txt
+FilterFile *.ps   /usr/share/dhelp/scripts/conv-pstotext  %f @/var/lib/dhelp/tmp/%B.txt
 
 IncludeFile HTML *.htm*
 IncludeFile Text *.txt


signature.asc
Description: This is a digitally signed message part


Bug#556509: Patch to address this issue

2011-09-17 Thread Georgios M. Zarkadas
tags 556509 + patch
tags 556509 + upstream
thanks

The attached patch seems to resolve the issue. The package built
succesfully after applying it, inside a pbuilder chroot with
EXTRAPACKAGES=binutils-gold, on an amd64 system.

The patch modifies just the Makefile. It is thus IMO an upstream issue,
so I took the liberty to add also the upstream tag. 

I attach the build log also, FYI. The lines demonstrating the correction
are 953-957 (the log grew somewhat big due to my chain of hook scripts).

regards
George Zarkadas

PS: As a sidenote, the only upstream-targeted OS that now does not
include the png library as a link target is OS-X; they may want to check
whether it is needed there also.
diff --git a/Makefile b/Makefile
index e7c4fff..e9e6b93 100644
--- a/Makefile
+++ b/Makefile
@@ -80,7 +80,7 @@ FRIBIDI_CFLAGS:=$(shell pkg-config --cflags fribidi)
 windows_ARCH_LINKS:=-lintl $(PNG) -lwinspool -lshlwapi $(FRIBIDI_LIB)
 osx_ARCH_LINKS:=$(PAPER_LIB) $(FRIBIDI_LIB)
 beos_ARCH_LINKS:=-lintl $(PNG) -lz -lbe -liconv $(FRIBIDI_LIB)
-linux_ARCH_LINKS:=$(PAPER_LIB) $(FRIBIDI_LIB)
+linux_ARCH_LINKS:=$(PAPER_LIB) $(PNG) $(FRIBIDI_LIB)
 ARCH_LINKS:=$($(OS)_ARCH_LINKS)
 
 windows_ARCH_HEADERS:=src/win32_print.h


tuxpaint_0.9.21-1_amd64.build.tar.bz2
Description: application/bzip-compressed-tar


signature.asc
Description: This is a digitally signed message part


Bug#641917: Correct version is 0.6.19

2011-09-17 Thread Georgios M. Zarkadas
reassign 641917 dhelp 0.6.19
thanks

The +nmu1 appearing in the package version at my first message 
is from the local version made to test this as well as #561883 and
#567889 patches. 
The patch is to the version 0.6.19 sources of the package.

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#556479: swish++: An updated and confirmed patch to build package with binutils-gold

2011-09-06 Thread Georgios M. Zarkadas
Package: swish++
Version: 6.1.5-2
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


Please disregard the comments of my previous message; while I had the gold 
linker in my system, the build chroot contained the old ld. Thus, the patch
supplied in bug #629813 does not alone fix this bug.

However, after getting gold inside the build chroot and trying to build the
package, I arrived at a patch that resolves this bug (#556479): supply two
more needed -l switches (-lgcc -lm) to the STDCXXLINK variable inside file
config.mk.

The combined patch that addresses both bugs (#556479, #629813) is attached.
The patch creates two more quilt patches and updates the series file.

I attach also the build log FYI. I will then subsequently merge the two bugs.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages swish++ depends on:
ii  libc6  2.11.2-10 Embedded GNU C Library: Shared lib
ii  libstdc++6 4.4.5-8   The GNU Standard C++ Library v3
ii  perl [perl5]   5.10.1-17squeeze2 Larry Wall's Practical Extraction 
ii  zlib1g 1:1.2.3.4.dfsg-3  compression library - runtime

swish++ recommends no packages.

Versions of packages swish++ suggests:
pn  antiword  none (no description available)
ii  poppler-utils [xpdf-utils]0.12.4-1.2 PDF utilitites (based on libpopple

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOZtfwAAoJEJWXIVmJ5BwWmFAH/jV+1qlDEBccwKEaaw6gT5N5
Jp/GjWrR/KMNN868GUZzn2TSDMWafbqNBOxUC9cr699o/VKKG82gDSBXxpZfodRa
td/gpNfaY/B4Ds0Gr/iE2XZ4YKmBzR6Z2X6HUYfDoTN9n+jw/w56KAnWI7WoK0hk
catrS7sDb0gcSBHRthvjipf1Bbdd8dX43jK2SRSJ8JqkG6LD7rhX8ofgofh0eVPs
2PVgL7aM+/xdZX4IPtOWOQnPcRJrloDJ0htRQQ4CRgocnGbnwbJ3XhOzUSILCrzW
cYm0a8OXrcVeMMIr6QGMdpjeY4zkoK7v9XQ3TC4PUmzg5Ttlb9NNtcdyLgBwK9M=
=ilNH
-END PGP SIGNATURE-
diff --git a/debian/patches/fix_ftbfs_gold_bug b/debian/patches/fix_ftbfs_gold_bug
new file mode 100644
index 000..8aaf3b1
--- /dev/null
+++ b/debian/patches/fix_ftbfs_gold_bug
@@ -0,0 +1,5 @@
+--- swish++-6.1.4.orig/config/config.mk
 swish++-6.1.4/config/config.mk
+@@ -186,1 +186,1 @@
+-STDCXXLINK:=	-lstdc++
++STDCXXLINK:=   -lstdc++ -lgcc -lm
diff --git a/debian/patches/include_cstddef_header b/debian/patches/include_cstddef_header
new file mode 100644
index 000..a59398b
--- /dev/null
+++ b/debian/patches/include_cstddef_header
@@ -0,0 +1,12 @@
+diff --git a/encoded_char.h b/encoded_char.h
+index 819c993..6b9a371 100644
+--- a/encoded_char.h
 b/encoded_char.h
+@@ -25,6 +25,7 @@
+ // standard
+ #include iterator
+ #include set
++#include cstddef
+ 
+ // local
+ #include iso8859-1.h
diff --git a/debian/patches/series b/debian/patches/series
index 6a80ecd..84d0831 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -9,3 +9,5 @@ fix_man_pages
 use_gcc_for_ld
 splitmail_junk_header
 fixincludes_gcc4.4
+include_cstddef_header
+fix_ftbfs_gold_bug


swish++_6.1.5-2_amd64.build.tar.bz2
Description: BZip2 compressed data


Bug#631609: jailer: Ubuntu fix confirmed for debian with changes to account for the packaging differences

2011-08-31 Thread Georgios M. Zarkadas
Package: jailer
Version: 0.4-16
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


I confirmed the bug on a clean debian unstable distribution and also that the
changes to debian/rules from the OP's patch indeed make the package to build.

However the OP's patch cannot apply as-is in Debian, due to the Ubuntu-specific
changes to debian/changelog and debian/control. These will have to be cleaned
up.

I supply as reference the diff of the changes made to debian packaging in order
for the package to build in a Debian build environment (debian/rules from the
OP's patch + one line in debian/control). The credits should of course go to
the OP.

regards
George Zarkadas


- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOXqFzAAoJEJWXIVmJ5BwWGbQIAJFFJD4zsKvugZnjh1DeoRSC
8vlkgCWTcHFHEt2wuBqkv5rxEpgF0/xDnI0cGyJ7uQ+mtdHcrrG4HJYqD5UoV/tO
nVlTQoHkkoOIekKB7CskZEl7fPNJQ1jrdvq2XXq8nyq1a6B9ws35HCuZObLLtDRz
wWFLeoPGs+/4rTHEKs9ujy23UiQiZDd2nQiVg46gWeQCkkA4USvW1T5sBCI6faEG
7ldcYRA1lVVSGIiKxanWCz7IM41M/efSYIdGB+HM4BvvXpEzKTTD3ipo+7rRpAx2
eZ57XtYaVBA/jfpgBj79ot31kPyV14f5dodHTx0kn0yPoQeYKgHc+Ba0DHRRAt0=
=UsGJ
-END PGP SIGNATURE-
--- a/debian/control
+++ b/debian/control
@@ -2,7 +2,7 @@
 Section: admin
 Priority: optional
 Maintainer: Javier Fernandez-Sanguino Pen~a j...@debian.org
-Build-Depends-Indep: debhelper ( 3.0.0), automake
+Build-Depends-Indep: debhelper ( 3.0.0), automake, dh-autoreconf
 Standards-Version: 3.6.1
 
 Package: jailer
--- a/debian/rules
+++ b/debian/rules
@@ -9,6 +9,7 @@
 build: build-stamp
 build-stamp:
dh_testdir
+   dh_autoreconf
./configure --prefix=/usr --mandir=\$${prefix}/share/man 
--infodir=\$${prefix}/share/info --sysconfdir=/etc
$(MAKE)
touch build-stamp
@@ -18,7 +19,7 @@ clean:
dh_testroot
rm -f build-stamp
-[ -f Makefile ]  $(MAKE) distclean 
-
+   dh_autoreconf_clean
dh_clean
 
 install: build


Bug#637743: Status update

2011-08-23 Thread Georgios M. Zarkadas
I just noticed that upstream has already incorporated a relevant patch
for its 0.7.10 release source tree. 

Thus, I will resubmit only a subset of this patch (namely the support
for connecting through local unix sockets and for the archive file
format) directly to upstream.

Therefore I guess this report should just be closed, unless there is
interest for a backport to the 0.7.9 version of the package (in which
case you can just notify me to prepare a patch based on the upstream
source, so that there is no need to change config files on upgrade).

regards
George Zarkadas


signature.asc
Description: This is a digitally signed message part


Bug#638919: backup-manager: Please use a single md5 sums file

2011-08-22 Thread Georgios M. Zarkadas
Package: backup-manager
Version: 0.7.9-3
Severity: wishlist
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Attached is a patch to replace the current scheme for storing md5 sums (one
file per day) with a single file. IMHO this simplifies the procedure and works
better with multiple configuration files, while allows easier scripting of
related tasks (for example a periodic check of archives' integrity) since all
the information is present in a central, fixed location.

The patch as is hardcodes the location of the file. It is trivial to make this
configurable, if desired; I will happily provide an updated patch in that case.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages backup-manager depends on:
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  findutils4.4.2-1+b1  utilities for finding files--find,
ii  ucf  3.0025+nmu1 Update Configuration File: preserv

backup-manager recommends no packages.

Versions of packages backup-manager suggests:
ii  anacron   2.3-14 cron-like program that doesn't go 
ii  backup-manager-doc0.7.9-3documentation package for Backup M
ii  dar   2.3.10-1+b1Disk ARchive: Backup directory tre
ii  dvd+rw-tools  7.1-6  DVD+-RW/R tools
ii  genisoimage   9:1.1.11-1 Creates ISO-9660 CD-ROM filesystem
ii  gettext-base  0.18.1.1-3 GNU Internationalization utilities
ii  libfile-slurp-perl.13-1  single call read  write file rout
pn  libnet-amazon-s3-perl none (no description available)
ii  openssh-client1:5.5p1-6+squeeze1 secure shell (SSH) client, for sec
ii  perl  5.10.1-17squeeze2  Larry Wall's Practical Extraction 
ii  wodim 9:1.1.11-1 command line CD/DVD writing tool
ii  zip   3.0-3  Archiver for .zip files

- -- debconf information excluded

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOUt05AAoJEJWXIVmJ5BwWYksIALyQKkTXhH/aODCW4h85kZCX
6stxO5kswBQZTfyAUgxS/8aO3tVuO0UFTrBbR7CMnHdVYH8HzMMgK/MPi7VaPsuW
ckhrGVEDGtnoSQeWA+bcEO8Zfgz2Y0Xkpld1LckzirXQzzPD2UxFxnG5yzw+VSLj
BYS/xxMS/FaAVxdhXv8YuoRz9LkeFfDDBwBKL/lLPnkf6M0mXsmynas1a6NLHQhK
ipdNh3Ksu9pWvwzBdRshDH0yEAjiKNxePoFbCLv97YagmwRtBMD3zkmeLIBVNx4Y
p+QBeYq20Nh5bu68TKLzKiV/9TnUVJcvUYuuarUIk9+ENUVtjWvKnfwSlNqU998=
=Hd7Z
-END PGP SIGNATURE-
--- a/backup-manager
+++ b/backup-manager
@@ -183,7 +183,9 @@
 source $libdir/sanitize.sh
 
 debug Initializing environment
-bm_init_env 
+bm_init_env
+# Use a single md5 file to store all archives.
+export MD5FILE=${BM_REPOSITORY_ROOT}/${BM_ARCHIVE_PREFIX}-hashes.md5
 
 debug Checking if logger is available
 check_logger
--- a/lib/actions.sh
+++ b/lib/actions.sh
@@ -49,13 +49,12 @@
 esac
 
 # Now make sure the md5 file is okay.
-md5file=$BM_REPOSITORY_ROOT/${BM_ARCHIVE_PREFIX}-${TODAY}.md5
-if [[ -e $md5file ]]  
+if [[ -e $MD5FILE ]]  
[[ $BM_REPOSITORY_SECURE = true ]]; then
-chown $BM_REPOSITORY_USER:$BM_REPOSITORY_GROUP $md5file ||
-warning Unable to change the owner of \\$md5file\.
-chmod $BM_ARCHIVE_CHMOD $md5file ||
-warning Unable to change file permissions of \\$md5file\.
+chown $BM_REPOSITORY_USER:$BM_REPOSITORY_GROUP $MD5FILE ||
+warning Unable to change the owner of \\$MD5FILE\.
+chmod $BM_ARCHIVE_CHMOD $MD5FILE ||
+warning Unable to change file permissions of \\$MD5FILE\.
 fi
 done
 }
--- a/lib/backup-methods.sh
+++ b/lib/backup-methods.sh
@@ -38,15 +38,13 @@
 echo $str ${md5hash})
 fi
 
-md5file=$BM_REPOSITORY_ROOT/${BM_ARCHIVE_PREFIX}-${TODAY}.md5
-
 # Check if the md5file contains already the md5sum of the file_to_create.
 # In this case, the new md5sum overwrites the old one.
-if grep $base $md5file /dev/null 21 ; then
-previous_md5sum=$(get_md5sum_from_file $base $md5file)
-sed -e /$base/s/$previous_md5sum/$md5hash/ -i $md5file
+if grep $base $MD5FILE /dev/null 21 ; then
+previous_md5sum=$(get_md5sum_from_file $base $MD5FILE)
+sed -e /$base/s/$previous_md5sum/$md5hash/ -i $MD5FILE
 else
-echo $md5hash  $base  $md5file
+echo $md5hash  $base  $MD5FILE
 fi
 
 # Now that the file is created, remove previous duplicates if exists...
--- a/lib/files.sh
+++ b/lib/files.sh
@@ -413,10 +413,9 @@
 error Unable to get date from file.
 

Bug#638920: backup-manager: Please provide an uploaded files database

2011-08-22 Thread Georgios M. Zarkadas
Package: backup-manager
Version: 0.7.9-3
Severity: wishlist
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Currently when more than one job is run at the same day, backup-manager 
uploads all archives of the day that exist in the repository in each job's
invocation. This results in extraneous uploads that consume bandwidth and 
may also introduce errors later on, although the archive was succesfully 
uploaded the first time.

It would be nice if backup-manager kept track of all succesfully uploaded
archives in a central database and consulted it to filter out unecessary
uploads. 

The attached patch does exactly that, using a flat text file as the database.

In addition, other administrative tasks could be facilitated by such a
database (I, for example, use dar and isolated catalogs. If the database
exists I could make a cron job to remove archives that are uploaded and
eplace them with symlinks to the catalogs, to save space).

The patch has been tested and is used already on my system without producing
errors so far.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages backup-manager depends on:
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  findutils4.4.2-1+b1  utilities for finding files--find,
ii  ucf  3.0025+nmu1 Update Configuration File: preserv

backup-manager recommends no packages.

Versions of packages backup-manager suggests:
ii  anacron   2.3-14 cron-like program that doesn't go 
ii  backup-manager-doc0.7.9-3documentation package for Backup M
ii  dar   2.3.10-1+b1Disk ARchive: Backup directory tre
ii  dvd+rw-tools  7.1-6  DVD+-RW/R tools
ii  genisoimage   9:1.1.11-1 Creates ISO-9660 CD-ROM filesystem
ii  gettext-base  0.18.1.1-3 GNU Internationalization utilities
ii  libfile-slurp-perl.13-1  single call read  write file rout
pn  libnet-amazon-s3-perl none (no description available)
ii  openssh-client1:5.5p1-6+squeeze1 secure shell (SSH) client, for sec
ii  perl  5.10.1-17squeeze2  Larry Wall's Practical Extraction 
ii  wodim 9:1.1.11-1 command line CD/DVD writing tool
ii  zip   3.0-3  Archiver for .zip files

- -- debconf information excluded

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOUt2XAAoJEJWXIVmJ5BwWHQ8IALobsYprBJu5FVBwcVOXQGkB
EmCgOxVd5ogiZQ2VVxKAQ9F5HBhX/JyjRxI3jxbc2gq3Dfn2FsyQFXpM6rGs9eIu
3yGTAIY1YtV2bHfeiUnl9hhQF3RQcuJ1nClLTw8cifWuGb+3qbmAWcSBicbJmYHV
KhDFncSiKHG7v4Q0uvGhMufAK31uJQQVachLHlex/9fTcZBPatHwFP39Jr6ZwZxA
j5/jwZx9SRDFAnlbOlHwfWjIYxdzrC5GLk95iF5rDYNv/J2+IUQMu6m6vywsskzI
kFXhDQoH9+Ki/rx+BvJ+unQK0TMXAbxyNwYH0gB7T22FJ4v+NoPhrGlCum5M/5E=
=oUkf
-END PGP SIGNATURE-
--- a/backup-manager-upload
+++ b/backup-manager-upload
@@ -105,6 +105,61 @@
}
 }
 
+# The idea behind BM_UPLOADED_ARCHIVES is to have a database of what archives
+# have been uploaded so far. This allows multiple execution of upload actions
+# within a day without resending all archives of the day from the beginning.
+
+# Add one file,host pair to $BM_UPLOADED_ARCHIVES database.
+# Called immediately *after* successful uploading of an archive.
+sub appendto_uploaded_archives($$)
+{
+my $file = shift;
+my $host = shift;
+unless ( defined $file and defined $host ) {
+print_error required args needed;
+return FALSE;
+}
+
+my $upload_fname = $ENV{BM_UPLOADED_ARCHIVES};
+unless ( defined $upload_fname ) {
+# Uncomment next line if you want the mandatory use
+# of BM_UPLOADED_ARCHIVES (ie always have it around).
+#print_error BM_UPLOADED_ARCHIVES is not defined;
+return FALSE;
+}
+
+# if $file already in database, append host to that line;
+# else append a lines $file $host to the end.
+
+my $io_error = 0;
+if ( ! system( grep -q \^$file \ $upload_fname ) ) {
+my $cmd = sed -i \s:^$file .*\$:\ $host:\ $upload_fname;
+$io_error = system($cmd);
+}
+elsif ( open(my $fh, , $upload_fname) ) {
+print($fh $file $host\n) or $io_error = 1;
+close $fh;
+}
+else {
+$io_error = 2;
+}
+if ( $io_error ) {
+print_error IO error: did not update $upload_fname with '$file 
$host';
+return FALSE;
+}
+
+return TRUE;
+}
+
+# Get all files of the specified date; filter the list through 
+# 

Bug#638803: backup-manager: If a backup crosses the day limit it won't be uploaded

2011-08-21 Thread Georgios M. Zarkadas
Package: backup-manager
Version: 0.7.9-3
Severity: normal
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Backup-manager calculates the date string of the archive name (the TODAY env
var) upon startup, then performs the backup, then runs backup-manager-upload
with a date argument of 'today' to upload the archive(s).

The 'today' argument results in recalculation of the date string on the fly
by bmu, at the time of invocation, ie just before the upload begins. Thus,
we have two calculations of the date part of the archive name separated by a
potentially significant time delay.

Therefore, if a backup starts at date X and finishes at date X + 1, it will
not get uploaded. I was actually hit by that before inspecting the code.

A patch is attached to address this issue. The patch replaces the 'today'
argument with the $TODAY variable, the one that its value is set during the
startup of backup-manager. 

I have tested the patch both by building the package and patching my system's
live configuration and it works (backups are uploaded). 

The patch has also the nice side effect of allowing the upload of archives of
any date, by writing 'export TODAY=...' in a configuration file, FOO, and then
issuing 'backup-manager --upload -c FOO', since backup-manager sets the TODAY
variable early, before reading the configuration file.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages backup-manager depends on:
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  findutils4.4.2-1+b1  utilities for finding files--find,
ii  ucf  3.0025+nmu1 Update Configuration File: preserv

backup-manager recommends no packages.

Versions of packages backup-manager suggests:
ii  anacron   2.3-14 cron-like program that doesn't go 
ii  backup-manager-doc0.7.9-3documentation package for Backup M
ii  dar   2.3.10-1+b1Disk ARchive: Backup directory tre
ii  dvd+rw-tools  7.1-6  DVD+-RW/R tools
ii  genisoimage   9:1.1.11-1 Creates ISO-9660 CD-ROM filesystem
ii  gettext-base  0.18.1.1-3 GNU Internationalization utilities
ii  libfile-slurp-perl.13-1  single call read  write file rout
pn  libnet-amazon-s3-perl none (no description available)
ii  openssh-client1:5.5p1-6+squeeze1 secure shell (SSH) client, for sec
ii  perl  5.10.1-17squeeze2  Larry Wall's Practical Extraction 
ii  wodim 9:1.1.11-1 command line CD/DVD writing tool
ii  zip   3.0-3  Archiver for .zip files

- -- debconf information excluded

- -- debsums errors found:
debsums: changed file /usr/share/backup-manager/actions.sh (from backup-manager 
package)
debsums: changed file /usr/share/backup-manager/files.sh (from backup-manager 
package)
debsums: changed file /usr/share/backup-manager/backup-methods.sh (from 
backup-manager package)
debsums: changed file /usr/share/backup-manager/upload-methods.sh (from 
backup-manager package)
debsums: changed file /usr/share/backup-manager/sanitize.sh (from 
backup-manager package)
debsums: changed file /usr/share/backup-manager/externals.sh (from 
backup-manager package)
debsums: changed file /usr/sbin/backup-manager (from backup-manager package)

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOUaHiAAoJEJWXIVmJ5BwWQMgIAI3ywdXCVZT1/PB5EpioLbWu
Lu1z1kW4dxeTyX/XciML7aVchhwSCOWp+ksS4JISHHz3Lh1UQoHhfBFfcHqY6bXj
wtl7Er6smBhxuNwCYO3lBrfhjJScPi6xZ7IYO1ilg67zGp1y3msWjx4y63ktsrgh
vl/l3hzoyEWDVjzgbdFdmrias+uxKmrZN7bwKksoQkBf3kmhYeFtGDe0Ak6H8ubj
V5/1UYYC3UQ4LWK6uG4D9HwTkBUYrKGekOJOXYsDzGGBSwnfTm1Dq/p+WaJsYtw6
1gUEZZ6kiIPa+KTIoJ/PW8F/0m23LDbqF5XI39WUrbC16SYDlju7Xp4XEoW3SaU=
=Zie9
-END PGP SIGNATURE-
--- a/lib/upload-methods.sh
+++ b/lib/upload-methods.sh
@@ -64,7 +64,7 @@
   -h=$bm_upload_hosts \
   -u=$BM_UPLOAD_SSH_USER \
   -d=$BM_UPLOAD_SSH_DESTINATION \
-  -r=$BM_REPOSITORY_ROOT today 2$logfile || 
+  -r=$BM_REPOSITORY_ROOT ${TODAY} 2$logfile || 
 error Error reported by backup-manager-upload for method \scp\, check 
\\$logfile\.
 rm -f $logfile
 }
@@ -101,7 +101,7 @@
  -u=$BM_UPLOAD_SSH_USER \
  -d=$BM_UPLOAD_SSH_DESTINATION \
  -r=$BM_REPOSITORY_ROOT \
- --gpg-recipient=$BM_UPLOAD_SSHGPG_RECIPIENT today 2$logfile|| 
+ --gpg-recipient=$BM_UPLOAD_SSHGPG_RECIPIENT ${TODAY} 2$logfile|| 
 error Error reported by 

Bug#608237: Same patch, new form

2011-08-18 Thread Georgios M. Zarkadas
I resubmit the same patch as in my previous message, but in a form
suitable for direct inclusion to the package's debian/patches directory.

I have confirmed that the patch is applied cleanly by making a test
build with both this and the patches from the #637743 bug report.

regards
George Zarkadas
--- a/lib/backup-methods.sh
+++ b/lib/backup-methods.sh
@@ -238,7 +238,9 @@
 target=$2
 debug __get_flags_relative_blacklist ($switch, $target)
 
-target=${target%/}
+if [ $target != / ]; then
+target=${target%/}
+fi
 blacklist=
 for pattern in $BM_TARBALL_BLACKLIST
 do
@@ -252,7 +254,13 @@
 # making a relative path...
 pattern=${pattern#$target}
 length=$(expr length $pattern)
-pattern=$(expr substr $pattern 2 $length)
+# for $target=/, no spare / is left at the beggining
+# after the # substitution; thus take substr from pos 1
+if [ $target != / ]; then
+pattern=$(expr substr $pattern 2 $length)
+else
+pattern=$(expr substr $pattern 1 $length)
+fi
 
 # ...and blacklisting it
 blacklist=$blacklist ${switch}${pattern}


signature.asc
Description: This is a digitally signed message part


Bug#637743: backup-manager: Please add a postgresql backup method

2011-08-13 Thread Georgios M. Zarkadas
Package: backup-manager
Version: 0.7.9-3
Severity: wishlist
Tags: upstream patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Attached you will find a patch (backup-method-postgresql.diff) to add a 
postgresql method
to backup manager. AFAIK PostgreSQL is the default DBMS installed by Debian 
Installer, thus
supplying such a backup method would be useful.

A second, optional, patch (rename-mysql-databases.diff) renames the dump 
created by mysql 
backup method for individual database(s) so that it contains the DBMS 
identifier, just 
like the postgres method supplied here does. It is not essential, but I thought 
it makes
sense, especially in view of possible future addition of other DBMS dumping 
methods.

The code uses the pg_dump and pg_dumpall utilities of postgresql to provide the 
ability to
dump specific database(s) or the entire postgres cluster. A new section was 
added to the
configuration template. Configuration directives were chosen in a way to be as 
much as
possible compatible with the existing ones of the MySQL backup method.

I have tested it with my (very basic at the moment) postgres setup and it seems 
to work ok:

- -- both the entire cluster and individual databases dumps are generated 
   (note: I have used only the local unix sockets method to connect in my tests)

- -- no errors from pd_dump, pg_dumpall appear in logs

I have also verified that the package builds if the patches are appended to the 
existing
debian/patches/series in the order given here (ie all patches succeed).

I release the code with the same license as backup-manager does.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages backup-manager depends on:
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  findutils4.4.2-1+b1  utilities for finding files--find,
ii  ucf  3.0025+nmu1 Update Configuration File: preserv

backup-manager recommends no packages.

Versions of packages backup-manager suggests:
ii  anacron   2.3-14 cron-like program that doesn't go 
ii  backup-manager-doc0.7.9-3documentation package for Backup M
ii  dar   2.3.10-1+b1Disk ARchive: Backup directory tre
ii  dvd+rw-tools  7.1-6  DVD+-RW/R tools
ii  genisoimage   9:1.1.11-1 Creates ISO-9660 CD-ROM filesystem
ii  gettext-base  0.18.1.1-3 GNU Internationalization utilities
ii  libfile-slurp-perl.13-1  single call read  write file rout
pn  libnet-amazon-s3-perl none (no description available)
ii  openssh-client1:5.5p1-6+squeeze1 secure shell (SSH) client, for sec
ii  perl  5.10.1-17squeeze2  Larry Wall's Practical Extraction 
ii  wodim 9:1.1.11-1 command line CD/DVD writing tool
ii  zip   3.0-3  Archiver for .zip files

- -- debconf information excluded


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJORxrpAAoJEJWXIVmJ5BwW6XAH/1Qaw4agOvpycstJFcxB0ysh
Rco1a5AoAaCboXv2+eumssg72XxSTfJEoVp+Ub6dgti6jtXYrVd9y8VSbrnloiuC
KNpj8s5UuZ0m0UXI4RUbNQkojp9PbUlM8iWM6f7w+53jXMo9g11Er89/xIshtkyd
3o+WJfpyReKCVXLJFVfQj3slpGLGwuTA22h5vHsHcpUGGrzfaKFu/81RhILPgwxa
0gRCtFCckRp4VCjDW0E5gOFb+tPaZb60UEi786sM5rp8rd872CJkWwEnp5mBw5fL
GKc5VQr97/lBC1hECBS5j93DRnWhQtthVghfy/6d7Iy/XNnRSTohjlUj92bMu6g=
=n41A
-END PGP SIGNATURE-
--- a/backup-manager.conf.tpl
+++ b/backup-manager.conf.tpl
@@ -72,6 +72,7 @@
 # - tarball
 # - tarball-incremental
 # - mysql
+# - postgresql
 # - svn
 # - pipe
 # - none
@@ -225,6 +226,73 @@
 export BM_MYSQL_EXTRA_OPTIONS=
 
 ##
+# Backup method: PostgreSQL
+#
+
+# This method is dedicated to PostgreSQL databases.
+# You should not use the tarball method for backing up database
+# directories or you may have corrupted archives (unless, -and
+# this is specific for PostgreSQL- you also continuously backup
+# the Write Ahead Log; see the PostgreSQL documentation for 
+# details).
+# Enter here the list of databases to backup.
+# Wildcard: __ALL__ (will dump all the databases in one archive)
+export BM_POSTGRESQL_DATABASES=__ALL__
+
+# The PostgreSQL user who is allowed to read every database of those
+# filled in BM_POSTGRESQL_DATABASES.
+export BM_POSTGRESQL_ADMINLOGIN=
+
+# The above user's password.
+# WARNING! Be sure to protect this file with strict permissions.
+export BM_POSTGRESQL_ADMINPASS=
+
+# The host 

Bug#387756: Patch to fix this bug and also #611720

2011-08-12 Thread Georgios M. Zarkadas
The attached patch:

-- moves the crack files in question from /var/run/Crack
to /var/lib/Crack
-- modifies the postrm script to purge /var/lib/Crack also

Thus it fixes both #387756 and #611720.

I didn't touched the debian/changelog; it will also need to be modified
to reflect the changes of the new package's revision.

regards
George Zarkadas
diff --git a/Crack.make b/Crack.make
index df6308f..201e0fd 100644
--- a/Crack.make
+++ b/Crack.make
@@ -141,7 +141,7 @@ fi
 # Hierarchy
 ###
 
-for dir in run run/bin
+for dir in lib lib/bin
 do
 	test -d $dir || mkdir $dir || exit 1
 done
@@ -172,7 +172,7 @@ fi
 echo Stamp: $bdname
 echo 
 
-bindir=run/bin/$bdname
+bindir=lib/bin/$bdname
 
 
 ###
diff --git a/Makefile b/Makefile
index 2c3b304..934cb60 100644
--- a/Makefile
+++ b/Makefile
@@ -39,17 +39,17 @@ clean:
 tidy: clean
 	-scripts/plaster
 	-scripts/fbmerge
-	-rm -f run/[DIEGTKM]*
-	-rm -f run/dict/gecos.*
-	-rm -f run/dict/gcperm.*
+	-rm -f lib/[DIEGTKM]*
+	-rm -f lib/dict/gecos.*
+	-rm -f lib/dict/gcperm.*
 
 spotless: tidy
-	-rm -rf run/bin
+	-rm -rf lib/bin
 
 rmdict:
-	-rm -rf run/dict
+	-rm -rf lib/dict
 
 clobber: spotless rmdict
-	-cat run/F-merged  F-merged.save
-	-rm -rf run
+	-cat lib/F-merged  F-merged.save
+	-rm -rf lib
 	( cd extra ; make clean )
diff --git a/debian/Crack.8 b/debian/Crack.8
index 1703d35..ffeb630 100644
--- a/debian/Crack.8
+++ b/debian/Crack.8
@@ -114,7 +114,7 @@ rule number, or to restart a run whilst skipping over a few rulesets, try:
 .SH FILES
 .I /usr/share/Crack
 Location of the Crack program and scripts.
-.I /var/run/Crack/
+.I /var/lib/Crack/
 Location for the temporary files used by Crack.
 
 .SH SEE ALSO
diff --git a/debian/crack-md5.dirs b/debian/crack-md5.dirs
index aaac1d9..0d7b6bd 100644
--- a/debian/crack-md5.dirs
+++ b/debian/crack-md5.dirs
@@ -2,4 +2,5 @@ usr/sbin
 usr/lib/Crack
 usr/share/Crack
 etc/Crack
-var/run/Crack/bin
+var/lib/Crack
+var/lib/Crack/bin
diff --git a/debian/crack.dirs b/debian/crack.dirs
index aaac1d9..0d7b6bd 100644
--- a/debian/crack.dirs
+++ b/debian/crack.dirs
@@ -2,4 +2,5 @@ usr/sbin
 usr/lib/Crack
 usr/share/Crack
 etc/Crack
-var/run/Crack/bin
+var/lib/Crack
+var/lib/Crack/bin
diff --git a/debian/postrm b/debian/postrm
index 6f0cf91..d495bc8 100644
--- a/debian/postrm
+++ b/debian/postrm
@@ -10,8 +10,10 @@ case $1 in
 ;;
 	purge)
 # If the user has run crack there is stuff in here
+	[ -d /var/lib/Crack ]  rm -rf /var/lib/Crack
 	[ -d /usr/share/Crack ]  rm -rf /usr/share/Crack
 	[ -d /etc/Crack ]  rm -rf /etc/Crack
+
 ;;
 
 
diff --git a/debian/rules b/debian/rules
index e09bcf5..51ff019 100755
--- a/debian/rules
+++ b/debian/rules
@@ -35,12 +35,12 @@ build-stamp: configure-stamp
 	chmod 755 Crack.make
 	./Crack.make 
 	# this is the crypt version, move it to other place
-	mv run/bin/debian/ run/bin/debian-crypt
+	mv lib/bin/debian/ lib/bin/debian-crypt
 	# To build the MD5 version
 	mv src/libdes src/libdes.orig
 	cd src/util  cp elcid.c elcid.c.orig  cp elcid.c,bsd elcid.c
 	./Crack.make
-	mv run/bin/debian/ run/bin/debian-md5
+	mv lib/bin/debian/ lib/bin/debian-md5
 	# Now restore it to the previous situation
 	mv src/libdes.orig src/libdes
 	cd src/util  cp elcid.c.orig elcid.c
@@ -59,7 +59,7 @@ clean:
 	 cd src/util  cp elcid.c.orig elcid.c ;\
 	fi
 	-$(MAKE) spotless
-	-rm -rf run
+	-rm -rf lib
 
 	dh_clean
 
@@ -82,16 +82,16 @@ install: build
 	cd  debian/crack-common/usr/share/Crack/  ln -s /etc/Crack conf
 	chmod a+x  debian/crack-common/usr/share/Crack/Crack  debian/crack-common/usr/share/Crack/Reporter
 	# For the DES version
-	cp -a run/bin/debian-crypt/* debian/crack/usr/lib/Crack
-	cd debian/crack/usr/share/Crack  ln -s /var/run/Crack run
+	cp -a lib/bin/debian-crypt/* debian/crack/usr/lib/Crack
+	cd debian/crack/usr/share/Crack  ln -s /var/lib/Crack run
 
-	cd debian/crack/var/run/Crack/bin/  ln -s /usr/lib/Crack debian
+	cd debian/crack/var/lib/Crack/bin/  ln -s /usr/lib/Crack debian
 	install -m755 debian/Crack debian/Crack-Reporter debian/crack/usr/sbin/
 	# For the MD5 version
-	cp -a run/bin/debian-md5/* debian/crack-md5/usr/lib/Crack
-	cd debian/crack-md5/usr/share/Crack  ln -s /var/run/Crack run
+	cp -a lib/bin/debian-md5/* debian/crack-md5/usr/lib/Crack
+	cd debian/crack-md5/usr/share/Crack  ln -s /var/lib/Crack run
 
-	cd debian/crack-md5/var/run/Crack/bin/  ln -s /usr/lib/Crack debian
+	cd debian/crack-md5/var/lib/Crack/bin/  ln -s /usr/lib/Crack debian
 	install -m755 debian/Crack debian/Crack-Reporter debian/crack-md5/usr/sbin/
 # Could change into this the previous statements:
 #	dh_movefiles


signature.asc
Description: This is a digitally signed message part


Bug#629813: Patch to fix

2011-08-10 Thread Georgios M. Zarkadas
The following patch fixes the issue (builds the package in an amd64):

==
diff --git a/encoded_char.h b/encoded_char.h
index 819c993..6b9a371 100644
--- a/encoded_char.h
+++ b/encoded_char.h
@@ -25,6 +25,7 @@
 // standard
 #include iterator
 #include set
+#include cstddef
 
 // local
 #include iso8859-1.h
==

I attach the build log for reference.

regards
George Zarkadas


swish++_6.1.5-2_amd64.build.tar.bz2
Description: application/bzip-compressed-tar


signature.asc
Description: This is a digitally signed message part


Bug#608237: First patch works with tar but not with dar (also supported by backup-manager)

2011-08-08 Thread Georgios M. Zarkadas
Package: backup-manager
Version: 0.7.9-3
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


The patch supplied in previous message leaves the paths in absolute form when
backup path is /. This works with tar, but not with dar which requires that
paths passed with the -P option be relative.

For example:

1tar -cf /mnt/backups/test.tar --exclude=/home --exclude=/var --exclude=/mnt \
  --exclude=/srv --exclude=/usr --exclude=/etc --exclude=/proc --exclude=/sys \
  --exclude=/dev --exclude=/tmp --exclude=/lib --exclude=/boot --verbose /

  works ok.

2dar --create /mnt/backups/test --fs-root=/ -P/home -P/var -P/mnt -P/srv \
  -P/usr -P/etc -P/proc -P/sys -P/dev -P/tmp -P/lib -P/boot --verbose

  produces error:

Reading config file: /etc/darrc
Parse error on command line (or included files): Cannot add an absolute path

I supply a new patch that makes path relative in all cases and so works with 
both tar and dar. See attached file.

regards
George Zarkadas


- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages backup-manager depends on:
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  findutils4.4.2-1+b1  utilities for finding files--find,
ii  ucf  3.0025+nmu1 Update Configuration File: preserv

backup-manager recommends no packages.

Versions of packages backup-manager suggests:
ii  anacron   2.3-14 cron-like program that doesn't go 
ii  backup-manager-doc0.7.9-3documentation package for Backup M
ii  dar   2.3.10-1+b1Disk ARchive: Backup directory tre
ii  dvd+rw-tools  7.1-6  DVD+-RW/R tools
ii  genisoimage   9:1.1.11-1 Creates ISO-9660 CD-ROM filesystem
ii  gettext-base  0.18.1.1-3 GNU Internationalization utilities
ii  libfile-slurp-perl.13-1  single call read  write file rout
pn  libnet-amazon-s3-perl none (no description available)
ii  openssh-client1:5.5p1-6+squeeze1 secure shell (SSH) client, for sec
ii  perl  5.10.1-17squeeze2  Larry Wall's Practical Extraction 
ii  wodim 9:1.1.11-1 command line CD/DVD writing tool
ii  zip   3.0-3  Archiver for .zip files

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOQEROAAoJEJWXIVmJ5BwWJJoIAN2W1EQUVxxynrRSfLrWejCu
Yyq9Bpdw4+qFVORki32UEFg8lIquPaiacikQEz5SN74CzNKLF83P2y1GkQ6xPj9s
7f0/5X4ZneDox1RnnI9L0FvMlX9q3LEZCzA8SXlTuAqpAhYKA2Nh4ic9rHahwouP
J+Zk63czcfqx2Lqm5ZmyX3+Pw9+LMFdZSWNcxiLjC/VFVmubxK9+4VyZijx6N6xW
rOnPBrT8c5KqmPSBXnD/Vdpb0kEnQF+UicnoPbDIC+CZrYdfR92cd6juNM9NWWc8
Uv+Hd5N+emDgTOmi/p7gO6loUeKZVKzUMgELSJRh/MeR/mysZDljMMtRqfAVyqE=
=4obj
-END PGP SIGNATURE-
--- backup-methods.sh.orig
+++ backup-methods.sh
@@ -238,7 +238,9 @@
 target=$2
 debug __get_flags_relative_blacklist ($switch, $target)
 
-target=${target%/}
+if [ $target != / ]; then
+target=${target%/}
+fi
 blacklist=
 for pattern in $BM_TARBALL_BLACKLIST
 do
@@ -252,7 +254,13 @@
 # making a relative path...
 pattern=${pattern#$target}
 length=$(expr length $pattern)
-pattern=$(expr substr $pattern 2 $length)
+# for $target=/, no spare / is left at the beggining
+# after the # substitution; thus take substr from pos 1
+if [ $target != / ]; then
+pattern=$(expr substr $pattern 2 $length)
+else
+pattern=$(expr substr $pattern 1 $length)
+fi
 
 # ...and blacklisting it
 blacklist=$blacklist ${switch}${pattern}


Bug#637152: pbuilder: Bogus 'E: Command line parameter [ ] is not a valid .dsc file name' error

2011-08-08 Thread Georgios M. Zarkadas
Package: pbuilder
Version: 0.200
Severity: normal
Tags: patch

When issuing DIST=... pdebuild [1] from within a package source directory
to start building it, /usr/lib/pbuilder/pbuilder-buildpackage outputs 
the above error and build fails.

After searching it, it appears that the following line (#62) in file
/usr/bin/pdebuild:

${PBUILDERROOTCMD} ${PDEBUILD_PBUILDER} --build 
${EXTRA_CONFIGFILE[@]/#/--configfile } --buildresult ${BUILDRESULT} 
--debbuildopts  --debbuildopts ${DEBBUILDOPTS} $@  
../${PKG_SOURCENAME}_${PKG_VERSION}.dsc

leads to the error. Specifically, it passes an empty argument through
$@, which then makes the conditional code (while [ -n $1 ]; do...
loop) in /usr/lib/pbuilder/pbuilder-checkparams [2] to break early,
thus supplying an empty dsc filename to
/usr/lib/pbuilder/pbuilder-buildpackage.

A patch is attached that resolved this issue with my configuration.
The configuration in use is also attached, for your reference.

NOTES:

[1] pdebuild is allowed to run without pwd; relevant lines of /etc/sudoers are: 

Cmnd_Alias PDEBUILD = /usr/bin/pdebuild,/usr/bin/debuild-pbuilder
DEVS HOST=(ALL) NOPASSWD: PDEBUILD

with DEVS being a User_Alias and HOST a Host_Alias.

[2] The debsums error below is due to the attached patch been applied to the 
file in question.

regards
George Zarkadas

-- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32 (SMP w/4 CPU cores; PREEMPT)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages pbuilder depends on:
ii  coreutils8.5-1   GNU core utilities
ii  debconf [debconf-2.0]1.5.36.1Debian configuration management sy
ii  debianutils  3.4 Miscellaneous utilities specific t
ii  debootstrap  1.0.26+squeeze1 Bootstrap a basic Debian system
ii  wget 1.12-2.1retrieves files from the web

Versions of packages pbuilder recommends:
ii  devscripts   2.10.69+squeeze1scripts to make the life of a Debi
ii  fakeroot 1.14.4-1Gives a fake root environment
ii  sudo 1.7.4p4-2.squeeze.2 Provide limited super user privile

Versions of packages pbuilder suggests:
ii  cowdancer 0.62+nmu2  Copy-on-write directory tree utili
ii  gdebi-core0.6.4  Simple tool to install deb files
pn  pbuilder-uml  none (no description available)

-- debconf information:
  pbuilder/mirrorsite: http://ftp.gr.debian.org/debian/
  pbuilder/nomirror:
  pbuilder/rewrite: false

-- debsums errors found:
debsums: changed file /usr/lib/pbuilder/pbuilder-checkparams (from pbuilder 
package)
--- /usr/lib/pbuilder/pbuilder-checkparams.orig	2011-07-27 00:39:13.744900089 +0300
+++ /usr/lib/pbuilder/pbuilder-checkparams	2011-07-27 00:46:34.139646901 +0300
@@ -41,7 +41,12 @@
 #option for user-mode-linux only.
 IGNORE_UMOUNT=
 
-while [ -n $1 ]; do 
+while [ $# -gt 0 ]; do
+#ignore empty arguments (for the case $@ is empty)
+if [ -z $1 ]; then
+	shift
+	continue
+fi
 case $1 in 
 	--basetgz) 
 	case ${PBCURRENTCOMMANDLINEOPERATION} in


pbuilder-config.tar.gz
Description: application/compressed-tar


signature.asc
Description: This is a digitally signed message part


Bug#630750: [Adduser-devel] Bug#630750: default NAME_REGEX value in /etc/adduser.conf is incorrect

2011-08-03 Thread Georgios M. Zarkadas
Hi,
thanks for the response, I give update information below.

Στις 24-07-2011, ημέρα Κυρ, και ώρα 14:22 +0100, ο/η Stephen Gran
έγραψε:
 ...
   # check user and group names also against this regular expression.
  - -#NAME_REGEX=^[a-z][-a-z0-9_]*\$
  +#NAME_REGEX=^[a-z][-a-z0-9_]*\$?$
 
 This can't be fixing it, the default is commented out.
 
The patch was made against the file (adduser.conf) that the package
ships. It is shipped with NAME_REGEX commented out (ie check disabled). 
But if you uncomment it, thus enabling the check, then the bug appears.

However, since the package ships the file with the test disabled by
default, I could not send a patch that enables the test by default.
Thus I sent a patch that corrects the bug when someone decides to enable
the test, which is IMHO the right way to patch in such a situation. 

  ...
  Γίνεται εγκατάσταση hal (0.5.14-3) ...
 ...
 I wonder if this is a locale specific problem.  Can you try (in a
 chroot, whatever), installing haldaemon with a default config for
 adduser to try to reproduce it?  I am curious if it will succeed if you
 then set LANG=C and install it again.  I suspect it will install.  If
 this is the case, can you let me know?

The default config of adduser will certainly install the package since
it has the (optional) test disabled. 
The problem appears when the test is enabled. And it is not
package-specific, nor locale-specific, based on the outcomes of the
following tests (full results are presented as annex to the end):

i)   add a user from the command line, with NAME_REGEX uncommented and
set to its default (shipped with the package) value: ^[a-z][-a-z0-9_]*\$

  i-a) with my system's default locale:

adduser foo -- error
adduser foo$-- ok
  
  i-b) with the C locale:

LC_ALL=C adduser bar-- error
LC_ALL=C adduser bar$   -- ok

ii)  grep a list of names with perl regular expressions enabled (since
adduser is written in perl and NAME_REGEX is a perl regex).
Although I present only the C locale case, the results where identical
in my system's default locale also.

  ii-a) with the default value: ^[a-z][-a-z0-9_]*\$-- error
  ii-b) with the value of the patch: ^[a-z][-a-z0-9_]*\$?$ -- ok

iii) do the same as ii using a perl program instead of grep. Same
remarks apply.

  iii-a) with the default value: ^[a-z][-a-z0-9_]*\$-- error
  iii-b) with the value of the patch: ^[a-z][-a-z0-9_]*\$?$ -- ok

As a final note, the default value (^[a-z][-a-z0-9_]*\$) simply does not
look ok when someone reads the code of the 'checkname' sub in adduser
executable (lines 864-886 as shipped with the adduser-3.112+nmu2
package).
The string in line 868 states that \$ is only allowed at the end of user
names for compatibility with Samba machines accounts. 
However, the default regex does not allow it as optional at the end but
instead it *requires* it to exist after at least one lowercase letter
and any combination of lowercase letters, numbers, dashes and
underscores.
This is certainly a bug, because apart from requiring an $ character to
appear in the name it also allows *any* other character after the $,
even invalid ones.
The ?$ that the patch adds at the end of the value makes the \$ optional
and ensures it will be (if present) the last character in the name (see
also Annex B at the end of the message).

I am at your disposal if you need additional information, the full
results of the tests follow.

regards
George Zarkadas

ANNEX A - FULL TEST RESULTS


i)   add a user from the command line, with NAME_REGEX uncommented and
set to its default (shipped with the package) value: ^[a-z][-a-z0-9_]*\$

EXPECTED OUTCOME: all tried user names should be accepted.

  i-a) with my system's default locale:

root@freedom:/etc# adduser foo
adduser: Please enter a username matching the regular expression
configured
via the NAME_REGEX configuration variable.  Use the `--force-badname'
option to relax this check or reconfigure NAME_REGEX.

root@freedom:/etc# adduser foo$
Adding user `foo$' ...
Adding new group `foo$' (1004) ...
Adding new user `foo$' (1004) with group `foo$' ...
Creating home directory `/home/foo$' ...
Copying files from `/etc/skel' ...
Enter new UNIX password: 
Retype new UNIX password: 
passwd: το συνθηματικό ενημερώθηκε επιτυχώς
Αλλαγή πληροφοριών χρήστη για τον foo$
Εισάγετε την νέα τιμή, ή πιέστε ENTER για την προκαθορισμένη
Πλήρες όνομα []: 
Αριθμός δωματίου []: 
Τηλέφωνο Εργασίας []: 
Τηλέφωνο Οικίας []: 
Άλλο []: 
Is the information correct? [Y/n] y
Adding new user `foo$' to extra groups ...
Adding user `foo$' to group `cdrom' ...
Adding user `foo$' to group `floppy' ...
Adding user `foo$' to group `audio' ...
Adding user `foo$' to group `video' ...
Adding user `foo$' to group `plugdev' ...
Adding user `foo$' to group `fuse' ...
Adding user `foo$' to group `users' ...

  i-b) with the C 

Bug#403467: bash: #403467 Unreproducible

2011-07-07 Thread Georgios M. Zarkadas
Package: bash
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


Unable to reproduce on my system.

The initial script executed without coredumping. A sample (first few lines) of
the log file that created in my /tmp is provided below:

unixy-backup.restore_metadata: NO_GLOBALS_ARGS=. .unixy-backup.meta_filelist
unixy-backup.restore_metadata: GLOBALS: 
LOGFILE=/tmp/unixy-backup.restore_metadata.giorgos.freedom.log FILELIST= 
unixy-backup.restore_metadata:  DEBUG=true VERBOSE=true VERIFY=true 
VERIFY_MORE=true
unixy-backup.restore_metadata:  SHOW_PROGRESS=true 
FILELIST_DELIMITER=--line-delimited
unixy-backup.restore_metadata:  CROSS_FILESYSTEM_BOUNDARIES=false
unixy-backup.restore_metadata: check_access: on_error=--die-on-error 
type=--file, mode=--read, remote_login=localhost, 
check_file=.unixy-backup.meta_filelist
unixy-backup.restore_metadata: Restoring file metadata using 
.unixy-backup.meta_filelist
unixy-backup.restore_metadata: .unixy-backup.meta_filelist is not NULL delimited
unixy-backup.restore_metadata: Restoring metadata for www-data www-data 755 
4096 . ./meta
unixy-backup.restore_metadata: File in .unixy-backup.meta_filelist not found: 
./meta
unixy-backup.restore_metadata: Restoring metadata for www-data www-data 775 
4096 . ./fom-serve-gaann
unixy-backup.restore_metadata: File in .unixy-backup.meta_filelist not found: 
./fom-serve-gaann
unixy-backup.restore_metadata: Restoring metadata for www-data www-data 775 
4096 . ./fom-serve-gaann/cache
unixy-backup.restore_metadata: File in .unixy-backup.meta_filelist not found: 
./fom-serve-gaann/cache
unixy-backup.restore_metadata: Restoring metadata for www-data www-data 664 164 
. ./fom-serve-gaann/cache/1.dep
unixy-backup.restore_metadata: File in .unixy-backup.meta_filelist not found: 
./fom-serve-gaann/cache/1.dep
unixy-backup.restore_metadata: Restoring metadata for www-data www-data 664 112 
. ./fom-serve-gaann/cache/2.dep
unixy-backup.restore_metadata: File in .unixy-backup.meta_filelist not found: 
./fom-serve-gaann/cache/2.dep

The smaller script in message #10 gave (also without coredumping) the 
following output (a single line of output):

/bin/bash

I believe that both scripts are not good test cases. The former is too complex
while the later misses information (what are the a b files needed there?).

IMHO if there are no new nailed-down test cases soon, the bug should be closed
as unreproducible.

regards
George Zarkadas


- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages bash depends on:
ii  base-files6.0squeeze2Debian base system miscellaneous f
ii  dash  0.5.5.1-7.4POSIX-compliant shell
ii  debianutils   3.4Miscellaneous utilities specific t
ii  libc6 2.11.2-10  Embedded GNU C Library: Shared lib
ii  libncurses5   5.7+20100313-5 shared libraries for terminal hand

Versions of packages bash recommends:
ii  bash-completion   1:1.2-3programmable completion for the ba

Versions of packages bash suggests:
ii  bash-doc  4.1-3  Documentation and examples for the

- -- Configuration Files:
/etc/bash.bashrc changed [not included]
/etc/skel/.bash_logout [Errno 13] Άρνηση πρόσβασης: u'/etc/skel/.bash_logout'
/etc/skel/.bashrc [Errno 13] Άρνηση πρόσβασης: u'/etc/skel/.bashrc'
/etc/skel/.profile [Errno 13] Άρνηση πρόσβασης: u'/etc/skel/.profile'

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOFpAcAAoJEJWXIVmJ5BwWal4IANEIFpSF116JO5JR0KRNcxsu
mcufhaVPhdWR95KPGWqQVr+TP/DrpHL9jq1R3hdCXGEg2nt6kYTDu5YEQgMezXHE
SOnMZUYsYOMw+x+8/IEsJDsciyM8utwvU0T2tbLJcLIpcV0AjBvydCpC25U/CFem
KXwkcQ+vPdyfXO0RsKdMLZVqOBGUxwUbpHwm5FLHytqsuiqQtmy1G4e5faCndhK1
lcmOeeCuFNw03BKuOddCb2YUhwk6FdoJc8oITt3wIAhLDv0lv10N8BPFhgwpYZVO
kURVfANDK5czI3Sc49Dln2BWrFWFb4PbqTe2k9h1JGfLOw/spLsRnmJ5Qb3kPJA=
=4Dg7
-END PGP SIGNATURE-



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#632868: base-files: derive PATH in /etc/profile from /etc/login.defs

2011-07-06 Thread Georgios M. Zarkadas
Package: base-files
Version: 6.0squeeze2
Severity: wishlist
Tags: patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


This is a solution to bug #571086 (now closed) which keeps /etc/login.defs as
the only place to set PATH, by computing the set there value on the fly using
only grep and coreutils (both essential packages and thus guaranteed to be 
always present).

The benefit of using it is that there is no need to sync the two PATH values;
any change to /etc/login.defs will be immediately applied to subsequent logins.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages base-files depends on:
ii  gawk [awk]1:3.1.7.dfsg-5 GNU awk, a pattern scanning and pr
ii  mawk [awk]1.3.3-15   a pattern scanning and text proces

base-files recommends no packages.

base-files suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOFIriAAoJEJWXIVmJ5BwWy4UH/RX8WAriNj/tVBXXdlr9rfOh
zvAkDynCJygZZuyKa44cA8taNzpiSnphQOgtoWufbxt6TNN637G037HJQmBGKkjy
ELzT3tcO3SpEIxX+m7+QgWhbc04Or/p96Khmy5xbyAqw1bUl9XK37EAlrj3j3IyN
8aAjbvnYaT3rASqX1oxlYSdCJBcaL0RATcBP9PMvmG8VJxWDDAbUV9PfGy5PG1HA
fSJtDYlcOEy71PqF5Ojr/Tbh9PUR5eVgn9sD/yqETgXgdW0O0y3VGF0j6u57eRfF
PF05H9wneOxt9hkfWEdykcUdtSDo9+/9f/lZ2yzntM482EGY4vBWSEOjL5Iy4so=
=/I6N
-END PGP SIGNATURE-
--- /etc/profile2010-08-06 20:42:50.0 +0300
+++ /etc/profile2011-07-06 18:40:59.187069301 +0300
@@ -1,10 +1,12 @@
 # /etc/profile: system-wide .profile file for the Bourne shell (sh(1))
 # and Bourne compatible shells (bash(1), ksh(1), ash(1), ...).
 
+# Presence of /etc/login.defs and ENV_xxx are mandatory. Thus do not check
+# or provide alternative action to avoid duplicating PATH definitions. 
 if [ `id -u` -eq 0 ]; then
-  PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
+   PATH=`grep 'ENV_SUPATH.*PATH=' /etc/login.defs | cut -d= -f2`
 else
-  PATH=/usr/local/bin:/usr/bin:/bin:/usr/local/games:/usr/games
+   PATH=`grep 'ENV_PATH.*PATH=' /etc/login.defs | cut -d= -f2`
 fi
 export PATH
 



Bug#632887: base-files: When bash is called as sh it should behave as sh

2011-07-06 Thread Georgios M. Zarkadas
Package: base-files
Version: 6.0squeeze2
Severity: normal
Tags: patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


Per bash manual: When bash is invoked with the name sh, it tries to mimic 
the startup behavior of historical versions of sh as closely as possible, 
while conforming to the posix standard as well.

However, the current conditional code in /etc/profile does not address this
and results in sourcing /etc/bash.bashrc even when bash is invoked as /bin/sh
(to try it, just prepend a line 'echo bash.bashrc' in /etc/bash.bashrc and
then issue a `sh --login' command, after of course ensuring that the /bin/sh
symlink points to /bin/bash).

A patch is attached that tests for this case (etc_profile-bash_as_sh.patch). 

Note that since bash enters posix mode after reading the startup files the 
only way to check is to examine the value of $BASH. This btw may also mean 
that any code inside the startup files which uses `shopt -oq posix' checks 
(such as bash_completion) is probably broken. Note also that there is no need
to check for the `bash --posix --login' case, since then bash does not read 
any startup files and only honors ENV variable. 

I don't know if letting /etc/bash.bashrc to be sourced even on the sh case has
been done deliberately, but IMHO is not good, since code in /etc/bash.bashrc 
typically assumes that the full bash featureset is available, which is this 
case is a wrong assumption.  

If that kind of site-wide customisation is desired for sh shells it would be 
better to be implemented as an `ENV=/etc/sh.shrc ; export ENV' sequence of 
commands for the general sh case (so that it is also available for dash and 
possibly other shells). I attach a supplementary patch for that, in case you
find this possibility of interest (etc_profile-sh.shrc.patch).

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash

Versions of packages base-files depends on:
ii  gawk [awk]1:3.1.7.dfsg-5 GNU awk, a pattern scanning and pr
ii  mawk [awk]1.3.3-15   a pattern scanning and text proces

base-files recommends no packages.

base-files suggests no packages.

- -- no debconf information

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEbBAEBAgAGBQJOFLJsAAoJEJWXIVmJ5BwW5lMH+NoI/zYEH8cdzyVLk3rEKQgS
oEgJsGAssGP0RVtQ3j6rNSe7ySth3k0dDkMYosi46z3bE+ovWEqIeP894GI37MBu
UlEDho49D48G4d7wlZm9IXP1EziT7Wn1pWYWCfRWZcIEp3qHm6OAsmSOWkpSh29k
k7PQuYreAaPTAymK3+lAFsSoE/BeNGOWXzOnUuCbxVORMYo3ZOpwTvG3Sa1ueqYC
gRz+r9IpwpG69ZN7FSx5fQGBmO3mJ5mko1xFA7njPOFJKOg/mS8Afb0A3q+OKlRQ
7LPnc6TJpDBnJnQ1GAhu+rclvJYeeEFNnc3FOyEZDy7Dn55mUTtNHGzADVDFnQ==
=8EBj
-END PGP SIGNATURE-
--- /etc/profile2010-08-06 20:42:50.0 +0300
+++ /etc/profile2011-07-06 18:40:59.187069301 +0300
@@ -9,11 +9,16 @@
 export PATH
 
 if [ $PS1 ]; then
-  if [ $BASH ]; then
-# The file bash.bashrc already sets the default PS1.
-# PS1='\h:\w\$ '
+  # When bash is called as sh it should behave as sh.
+  # Note that since bash enters posix mode after reading startup files,
+  # the only test that can be applied here is the value of $BASH.
+  if [ $BASH ]  [ $BASH != /bin/sh ]; then
 if [ -f /etc/bash.bashrc ]; then
+  # The file bash.bashrc already sets the default PS1.
   . /etc/bash.bashrc
+else
+  # Get a sane default if bash.bashrc does not exist. 
+  PS1='\h:\w\$ '
 fi
   else
 if [ `id -u` -eq 0 ]; then

--- /etc/profile2010-08-06 20:42:50.0 +0300
+++ /etc/profile2011-07-06 18:40:59.187069301 +0300
@@ -22,6 +22,17 @@
   PS1='$ '
 fi
   fi
+
+  # Support a similar to bash.bashrc site-wide customisation
+  # for interactive shells for the plain bourne shell.
+  # We put it out of the bash/sh conditional so that it is available
+  # to users with a bash login shell when starting an interactive sh
+  # subshell.
+
+  if [ -f /etc/sh.shrc ]; then
+ENV=/etc/sh.shrc
+export ENV
+  fi
 fi
 
 # The default umask is now handled by pam_umask.

--- /dev/null   2011-07-06 18:40:59.187069301 +0300
+++ /etc/sh.shrc2011-07-06 18:40:59.187069301 +0300
@@ -1,0 +1,4 @@
+# System-wide .shrc file for interactive sh(1) shells.
+
+# To enable the settings / commands in this file for login shells as well,
+# set ENV=/etc/sh.shrc ; export ENV in /etc/profile.



Bug#613011: global: Bugs re-confirmed on a debian squeeze system

2011-06-27 Thread Georgios M. Zarkadas
Package: global
Version: 5.7.1-1
Severity: normal

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

I just confirmed that both bugs exist and that both patches 
(for /usr/lib/cgi-bin/gsearch.cgi and for /usr/bin/htconfig) 
are working on a debian/squeeze system.

(NB: The debsums error is due to the patched htconfig)

regards

- -- System Information:
Debian Release: 6.0.2
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages global depends on:
ii  libc6 2.11.2-10  Embedded GNU C Library: Shared lib

global recommends no packages.

Versions of packages global suggests:
ii  apache2-mpm-prefork [h 2.2.16-6+squeeze1 Apache HTTP Server - traditional n
pn  doxygennone(no description available)
ii  epiphany-browser [www- 2.30.6-1  Intuitive GNOME web browser
ii  iceweasel [www-browser 3.5.16-7  Web browser based on Firefox
ii  id-utils   4.2-1 Fast, high-capacity, identifier da
ii  w3m [www-browser]  0.5.2-9   WWW browsable pager with excellent

- -- no debconf information

- -- debsums errors found:
debsums: changed file /usr/bin/htconfig (from global package)

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJOCQ2uAAoJEJWXIVmJ5BwWzSsH/1vg1Y2RrF4ehMCl1kswgafx
bo7BUyhGe3EqYoqvcxz+AWebl9MkP6pAP3vlQLYa0wnsz6YHJ9RalHZj93vIvJB5
Yv3hFWczvp0eJT/Ngmvven6BgaHaDRJ9tHnNX09jvF66TqgxdnxWZ9vejbNiUe87
DQEzba0sUGdkrSUVbYVT9fNB9hwGN3ohtEi8uI+A4Zr7xg/YxOvfZIQVJqZQuAY3
HHvw5SMaJ+puDQ5ff8U3eLugxLQo4Ve/QdxLjYmhcRnL6kndcdem8nfPkJu+Z+sO
5vDsucbd9BihfccgyDvg1KIkd5K0X9G4fzGH2t+EV7nsXJ9veDXfbae4deeMztc=
=QUcS
-END PGP SIGNATURE-



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#630750: default NAME_REGEX value in /etc/adduser.conf is incorrect

2011-06-16 Thread Georgios M. Zarkadas
Package: adduser
Version: 3.112+nmu2
Severity: normal
Tags: patch

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1


The default value of variable NAME_REGEX shipped within the package's file 
/usr/share/adduser/adduser.conf (which is copied upon installation to
etc/adduser.conf) is incorrect.

current value / should-be value (the patch - append ?$ to it):

- --- adduser.conf  2011-06-17 00:51:05.718593580 +0300
+++ adduser.conf2011-06-17 00:51:32.282093072 +0300
@@ -82,4 +82,4 @@
 
 
 # check user and group names also against this regular expression.
- -#NAME_REGEX=^[a-z][-a-z0-9_]*\$
+#NAME_REGEX=^[a-z][-a-z0-9_]*\$?$

The problem realised when tried to install hal (which installs user haldaemon)
with the regex enabled and it failed to install with the following message:

- ---
Γίνεται εγκατάσταση hal (0.5.14-3) ...
adduser: Please enter a username matching the regular expression configured
via the NAME_REGEX configuration variable.  Use the `--force-badname'
option to relax this check or reconfigure NAME_REGEX.
dpkg: σφάλμα στην επεξεργασία του hal (--configure):
 η υποδιεργασία installed post-installation script επέστρεψε κατάσταση λάθους 1 
- ---

After checking the postinst script of hal package I didn't found anything 
suspicious in its adduser invocation, nor on the tried out username (haldaemon)
so I started to study the adduser source code. 

Seeing the 'checkname' routine and the initial test expression, I finally came
up with the patch which was tried and hal installation succeeded.

regards
George Zarkadas

- -- System Information:
Debian Release: 6.0.1
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'proposed-updates'), (500, 
'stable'), (450, 'testing-proposed-updates'), (450, 'testing'), (400, 
'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/4 CPU cores)
Locale: LANG=el_GR.utf8, LC_CTYPE=el_GR.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages adduser depends on:
ii  debconf [de 1.5.36.1 Debian configuration management sy
ii  passwd  1:4.1.4.2+svn3283-2+squeeze1 change and administer password and
ii  perl-base   5.10.1-17minimal Perl system

adduser recommends no packages.

Versions of packages adduser suggests:
ii  liblocale-gettext-perl1.05-6 Using libc functions for internati
ii  perl-modules  5.10.1-17  Core Perl modules

- -- debconf information:
  adduser/homedir-permission: true

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.10 (GNU/Linux)

iQEcBAEBAgAGBQJN+n7xAAoJEJWXIVmJ5BwWjvYIAKJu5Jojf+N4hTijmTr7fsFT
PnbNa5WEpAPWsP38zP+vo/V/CPHP9+tlm0HlspXdWVcG9IsW7FD0CbrI9yQqibSk
9cDpOcNnDSuJhjTrRlKN47m9pDcn0Soj+1GalSS55Th54f2vQEHv/xv60IhGHc0h
A7TATZCrGQdmuLWe/vA5fFTgz6obcqydExStceq1WhfLdUfn1c/nMkrqKRuxD4zA
bUj0v/ZBADxIwv+reFMBt22JD/RbBp4zYbqMPvvRuC9zbGXKSJoasPmKgbd+nN9l
+DBy/hD74gOFz9VUw9bRdt2XTWIzuwPZ4lWq1vvsiDHOgZkdUSpnX51j4H+5AGY=
=kxu5
-END PGP SIGNATURE-



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org