Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-05-14 Thread rhatto
Em Sun, May 13, 2012 at 02:38:34AM +0200, intrigeri escreveu: Hi, I'm confused about the status of this specific bug: it's tagged as fixed-upstream in the Debian BTS, but last message from Paul reads With the patch it's even worse. I have just merged half a ton of rhatto's bugfixes

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-05-14 Thread Paul van Tilburg
On Mon, May 14, 2012 at 10:40:48AM -0300, rhatto wrote: Em Sun, May 13, 2012 at 02:38:34AM +0200, intrigeri escreveu: I have just merged half a ton of rhatto's bugfixes branches into upstream master, may someone please check if the bug is still present in the resulting Git upstream master

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-05-14 Thread intrigeri
Hi, Paul van Tilburg wrote (14 May 2012 13:54:28 GMT) : I can say at least, that the rsync handler I started using at some point started to respect my setting: rsync_options = -avF --delete I can see that it passes both options. Thank you! So this specific bug is rightfully marked as

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-05-12 Thread intrigeri
Hi, I'm confused about the status of this specific bug: it's tagged as fixed-upstream in the Debian BTS, but last message from Paul reads With the patch it's even worse. I have just merged half a ton of rhatto's bugfixes branches into upstream master, may someone please check if the bug is still

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-16 Thread rhatto
Em Sun, Apr 15, 2012 at 10:26:27PM +0200, intrigeri escreveu: Paul van Tilburg wrote (15 Apr 2012 14:16:39 GMT) : Ok, yes. You must understand that this is already the 6th patch applied to /usr/share/backupninja/rsync and I am starting to loose track of what I am doing. rhatto, I think

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-15 Thread Paul van Tilburg
On Fri, Apr 13, 2012 at 10:09:08AM -0300, rhatto wrote: Em Fri, Apr 13, 2012 at 09:40:09AM +0200, Paul van Tilburg escreveu: On Thu, Apr 12, 2012 at 09:34:42PM -0300, rhatto wrote: Could you try the patch below? but rsync doesn't seem to use it for some reason. I see it backing up

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-15 Thread rhatto
Em Sun, Apr 15, 2012 at 04:00:02PM +0200, Paul van Tilburg escreveu: On Fri, Apr 13, 2012 at 10:09:08AM -0300, rhatto wrote: Em Fri, Apr 13, 2012 at 09:40:09AM +0200, Paul van Tilburg escreveu: On Thu, Apr 12, 2012 at 09:34:42PM -0300, rhatto wrote: Could you try the patch below?

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-15 Thread Paul van Tilburg
On Sun, Apr 15, 2012 at 11:10:01AM -0300, rhatto wrote: Em Sun, Apr 15, 2012 at 04:00:02PM +0200, Paul van Tilburg escreveu: I am unsure if the patch is meant to be on top of the previous on, or instead of the previous one. It's an incremental patch, so you apply atop of the previous.

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-15 Thread intrigeri
Hi, Paul van Tilburg wrote (15 Apr 2012 14:16:39 GMT) : Ok, yes. You must understand that this is already the 6th patch applied to /usr/share/backupninja/rsync and I am starting to loose track of what I am doing. rhatto, I think you'd better provide Paul with patches against the unmodified

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-13 Thread Paul van Tilburg
On Thu, Apr 12, 2012 at 09:34:42PM -0300, rhatto wrote: Em Mon, Apr 09, 2012 at 02:13:01PM +0200, Paul van Tilburg escreveu: This partly fixes it. Indeed, the --delete gets passed on now, but still expansion happens. But the expansion still happens (irregularly), so '/*bin' is expanded to

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-13 Thread rhatto
Em Fri, Apr 13, 2012 at 09:40:09AM +0200, Paul van Tilburg escreveu: On Thu, Apr 12, 2012 at 09:34:42PM -0300, rhatto wrote: Em Mon, Apr 09, 2012 at 02:13:01PM +0200, Paul van Tilburg escreveu: This partly fixes it. Indeed, the --delete gets passed on now, but still expansion happens.

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-12 Thread rhatto
Em Mon, Apr 09, 2012 at 02:13:01PM +0200, Paul van Tilburg escreveu: This partly fixes it. Indeed, the --delete gets passed on now, but still expansion happens. But the expansion still happens (irregularly), so '/*bin' is expanded to '/bin' and '/sbin', but '/home/*/.cache' isn't, and

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-04-09 Thread Paul van Tilburg
Hi, An update about this bug. On Sun, Feb 26, 2012 at 02:11:16PM +0100, intrigeri wrote: Paul van Tilburg wrote (07 Jan 2012 11:14:54 GMT) : It applies as is onto 0.9.10, but it did not help. Given the above rsync_options, it runs: Debug: nice -n 0 /usr/bin/rsync -avF --exclude=/bin

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-02-26 Thread intrigeri
Hi Paul, Paul van Tilburg wrote (07 Jan 2012 11:14:54 GMT) : It applies as is onto 0.9.10, but it did not help. Given the above rsync_options, it runs: Debug: nice -n 0 /usr/bin/rsync -avF --exclude=/bin --exclude=/sbin [...] So, the --delete got dropped. Current upstream Git (at commit

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-01-23 Thread Paul van Tilburg
Hello intrigeri, On Thu, Jan 05, 2012 at 03:36:23PM +0100, intrigeri wrote: Paul van Tilburg wrote (05 Jan 2012 12:43:30 GMT) : I tried to change the rsync options of the handler by setting: rsync_options = -avF --delete when I noticed that --delete didn't make it to the commandline.

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-01-05 Thread Paul van Tilburg
Package: backupninja Version: 0.9.10-1 Severity: normal Hi! I tried to change the rsync options of the handler by setting: rsync_options = -avF --delete when I noticed that --delete didn't make it to the commandline. I tried to set rsync_options = -avF --delete, but that didn't work either

Bug#654708: backupninja: rsync handler only respects the first rsync option

2012-01-05 Thread intrigeri
Hi Paul, Paul van Tilburg wrote (05 Jan 2012 12:43:30 GMT) : I tried to change the rsync options of the handler by setting: rsync_options = -avF --delete when I noticed that --delete didn't make it to the commandline. May you please try to apply the following patch?