Discussion:
[BackupPC-devel] BackupPC 4.1.0 released
Craig Barratt
2017-03-25 01:09:47 UTC
Permalink
BackupPC community,

I'm happy to announce that BackupPC 4.1.0 has been released on Github
<https://github.com/backuppc/backuppc/releases>. There are also new
versions of BackupPC::XS
<https://github.com/backuppc/backuppc-xs/releases> (0.53)
and rsync-bpc <https://github.com/backuppc/rsync-bpc/releases> (3.0.9.6).
on Github.

There are a few new features and important bug fixes, including:

- BackupPC_migrateV3toV4 optionally allows old 3.x backups to be
migrated to 4.x format, eliminating the hardlinks used by 3.x. (BackupPC
4.x can view/browse/restore 3.x backups, so it's not necessary to run
BackupPC_migrateV3toV4)
- modern CSS (from Ernesto Carrea)
- fixes to rsync restore (from Stephen Joyce)
- fixes to exponential backup expiry (from Alexander Moisseev)
- added template systemd files
- configure.pl adds a --config-only option for package builders (from
Alexander Moisseev)

If you are using 4.0.0, upgrading is strongly recommended.

After installing those two packages, BackupPC 4.1.0 can be installed from
the tar ball with:

tar zxf BackupPC-4.1.0.tar.gz
cd BackupPC-4.1.0
perl configure.pl

See the README.md, ChangeLog and doc/BackupPC.html
<http://backuppc.sourceforge.net/BackupPC-4.0.0.html> files for more
information. An overview of the changes is below.

Thanks for to everyone who provided fixes and reported issues.

Enjoy!

Craig

* Merged pull requests: #17, #44, #59, #60, #61, #62,#68, #69, #73, #74

* Fixed certain rsync restores, based on patch from Stephen Joyce.

* Improvements to Gentoo init.d script (#69) from sigmoidal.

* Fixed exponential backup expiry algorithm, submitted by Alexander
Moisseev (#17).

* Added --config-only option to configure.pl, from Alexander Moisseev (#62).

* Ensure $? is 0 in bin/BackupPC_dump UserCommandRun() if no command is run,
reported and proposed fix by stirab (issue #67).

* configure.pl replaces SourceForge links in $Conf{CgiNavBarLinks} with
new Github links.

* Added BackupPC_migrateV3toV4 that migrates old V3 backup storage to
V4, eliminating hardlinks. The forward merging of V3 incrementals is
not changed; just the backup tree is updated to use V4 attrib files, and
the linked V3 files are removed. Thanks to Michael Huntley for testing.

* Renamed init.d to systemd, added systemd/src/backuppc.service template

* Replaced CSS file with new BackupPC_stnd.css from Ernesto Carrea
(#59,#73).
Renamed old ones to conf/BackupPC_retro_v2.css and
conf/BackupPC_retro_v3.css.

* Updated lib/BackupPC/DirOps.pm with more robust checking that IO::Dirent
works. Matches similar changes to 3.x that didn't make it into 4.x.
Fixes issue #56.

* BackupPC_refCountUpdate handles case of empty backups better. It creates
a noPoolCntOk file if there are legitimately no poolCnt files after
running. Reported by Alexander Kobel.

* Fixed umask() in child processes (issue #58) reported by Alexander
Moisseev.
Richard Shaw
2017-03-25 13:06:31 UTC
Permalink
Craig,

Thanks for the new release! I'm working on updating the Fedora package now.

One question though... How hard would it be to add a "--ignore-install"
option?

I've been trying to test my package on my desktop but I also build the
package on my desktop so I keep having to uninstall the package or rename
config.pl so it doesn't find it during the build.

Thanks,
Richard
Alexander Moisseev
2017-03-25 14:11:29 UTC
Permalink
Post by Richard Shaw
Craig,
Thanks for the new release! I'm working on updating the Fedora package now.
One question though... How hard would it be to add a "--ignore-install" option?
I've been trying to test my package on my desktop but I also build the package on my desktop so I keep having to uninstall the package or rename config.pl <http://config.pl> so it doesn't find it during the build.
Thanks,
Richard
Richard,
If I understand your problem correctly, you can use "--dest-dir" option to build the package in separate staging directory.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-26 01:18:03 UTC
Permalink
Do you mean an option to configure.pl to ignore an existing install?

Have you tried doing this:

./configure.pl --config-path ''

(that's 2 single quotes to create an empty --config-path). It should
prompt for the config.pl path, and just hit enter.

Craig
Post by Richard Shaw
Craig,
Thanks for the new release! I'm working on updating the Fedora package now.
One question though... How hard would it be to add a "--ignore-install"
option?
I've been trying to test my package on my desktop but I also build the
package on my desktop so I keep having to uninstall the package or rename
config.pl so it doesn't find it during the build.
Thanks,
Richard
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Richard Shaw
2017-03-26 02:15:36 UTC
Permalink
On Sat, Mar 25, 2017 at 8:18 PM, Craig Barratt <
Post by Craig Barratt
Do you mean an option to configure.pl to ignore an existing install?
./configure.pl --config-path ''
(that's 2 single quotes to create an empty --config-path). It should
prompt for the config.pl path, and just hit enter.
Hmm... That would be a nogo since package building can't be interactive...

Thanks,
Richard
Bill Broadley
2017-03-26 02:35:37 UTC
Permalink
I upgraded two backuppc servers (both ubuntu 14.04) so far with:
1) cpan install BackupPC::XS
2) manual install of rsync-bpc (./configure --prefix=/opt/pkg/rsync-
bpc-<version>; make; make install
3) then just ran the backuppc configure script (and mention the path for
rsync-bpc)

It "just worked", except the ubuntu init script has:
grep -q IncrLevel $CONF || BAD=1

Which triggers (for those googleing the error):
BackupPC cannot be started because important parameters are missing from
config.pl. If you just upgraded BackupPC, please update /etc/backuppc
/config.pl

Took me awhile to find because it's not in dmesg, syslog, or stdout/stderr for
the daemon.

I added to the config.pl:
'IncrLevel' => '1',

Started up. Seems happy, did a test restore, and am now doing a migrate to see
how that impacts performance, pool size, and number of inodes.

On a 2.7TB pool so far it's around 30% more inodes and 1% more space, but it's
only processes less than half the hosts. I'm also rather curious about the time
to backup for 3.x vs 4.x.









------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-26 03:59:49 UTC
Permalink
Yes, IncrLevel is no longer used in 4.x. But rather than add a bogus
setting to config.pl, it would be better to fix the init script so that it
doesn't enforce a condition that is no longer true for 4.x.

Craig
Post by Bill Broadley
1) cpan install BackupPC::XS
2) manual install of rsync-bpc (./configure --prefix=/opt/pkg/rsync-
bpc-<version>; make; make install
3) then just ran the backuppc configure script (and mention the path for
rsync-bpc)
grep -q IncrLevel $CONF || BAD=1
BackupPC cannot be started because important parameters are missing from
config.pl. If you just upgraded BackupPC, please update /etc/backuppc
/config.pl
Took me awhile to find because it's not in dmesg, syslog, or stdout/stderr for
the daemon.
'IncrLevel' => '1',
Started up. Seems happy, did a test restore, and am now doing a migrate to see
how that impacts performance, pool size, and number of inodes.
On a 2.7TB pool so far it's around 30% more inodes and 1% more space, but it's
only processes less than half the hosts. I'm also rather curious about the time
to backup for 3.x vs 4.x.
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-26 03:58:09 UTC
Permalink
I'm not sure how you are using configure.pl to build a package.

In any case, what I meant was you should try adding --config-path '' to the
existing configure.pl arguments. Is that enough to have it ignore any
existing installation?w

Craig
Post by Richard Shaw
On Sat, Mar 25, 2017 at 8:18 PM, Craig Barratt <
Post by Craig Barratt
Do you mean an option to configure.pl to ignore an existing install?
./configure.pl --config-path ''
(that's 2 single quotes to create an empty --config-path). It should
prompt for the config.pl path, and just hit enter.
Hmm... That would be a nogo since package building can't be interactive...
Thanks,
Richard
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Kenneth Porter
2017-03-26 04:16:19 UTC
Permalink
--On Saturday, March 25, 2017 9:58 PM -0700 Craig Barratt
Post by Craig Barratt
I'm not sure how you are using configure.pl to build a package.
In any case, what I meant was you should try adding --config-path '' to
the existing configure.pl arguments. Is that enough to have it ignore any
existing installation?
Presumably the package should be built with its final config path,
/etc/BackupPC. But building the package might be done on a machine with a
working BackupPC, where the existing configuration should be ignored and
the package treated as a virgin installation. So it's necessary to tell the
build system the correct path but to ignore anything currently present at
that path.



---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-26 05:19:14 UTC
Permalink
In looking a the code, if config-path is specified then any current config
file is ignored.

If you specify dest-dir then all of the output files are put below there.

So if you specify both --config-path '' and --dest-dir, I believe any
existing install should be ignored.

Craig
Post by Kenneth Porter
--On Saturday, March 25, 2017 9:58 PM -0700 Craig Barratt
Post by Craig Barratt
I'm not sure how you are using configure.pl to build a package.
In any case, what I meant was you should try adding --config-path '' to
the existing configure.pl arguments. Is that enough to have it ignore
any
Post by Craig Barratt
existing installation?
Presumably the package should be built with its final config path,
/etc/BackupPC. But building the package might be done on a machine with a
working BackupPC, where the existing configuration should be ignored and
the package treated as a virgin installation. So it's necessary to tell the
build system the correct path but to ignore anything currently present at
that path.
---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Richard Shaw
2017-03-26 14:20:44 UTC
Permalink
On Sun, Mar 26, 2017 at 12:19 AM, Craig Barratt <
Post by Craig Barratt
In looking a the code, if config-path is specified then any current config
file is ignored.
If you specify dest-dir then all of the output files are put below there.
So if you specify both --config-path '' and --dest-dir, I believe any
existing install should be ignored.
Don't seem to work, it's already in there:

%{__perl} configure.pl \
--batch \
--backuppc-user=backuppc \
--dest-dir %{buildroot} \
--config-dir %{_sysconfdir}/%{name}/ \
--config-override XferMethod=rsync \
--config-override CgiURL=http://localhost/%{name} \
--config-override ClientNameAlias=0 \
--cgi-dir %{_libexecdir}/%{name} \
--data-dir %{_localstatedir}/lib/%{name}/ \
--html-dir %{_datadir}/%{name}/html/ \
--html-dir-url /%{name}/images \
--install-dir %{_datadir}/%{name} \
--log-dir %{_localstatedir}/log/%{name} \
--hostname localhost \
--no-set-perms \
--uid-ignore

Thanks,
Richard
Alexander Moisseev
2017-03-26 15:08:40 UTC
Permalink
In looking a the code, if config-path is specified then any current config file is ignored.
If you specify dest-dir then all of the output files are put below there.
So if you specify both --config-path '' and --dest-dir, I believe any existing install should be ignored.
%{__perl} configure.pl <http://configure.pl> \
--batch \
--backuppc-user=backuppc \
--dest-dir %{buildroot} \
--config-dir %{_sysconfdir}/%{name}/ \
--config-override XferMethod=rsync \
--config-override CgiURL=http://localhost/%{name} \
--config-override ClientNameAlias=0 \
--cgi-dir %{_libexecdir}/%{name} \
--data-dir %{_localstatedir}/lib/%{name}/ \
--html-dir %{_datadir}/%{name}/html/ \
--html-dir-url /%{name}/images \
--install-dir %{_datadir}/%{name} \
--log-dir %{_localstatedir}/log/%{name} \
--hostname localhost \
--no-set-perms \
--uid-ignore
But you have specified --config-dir instead of --config-path .

Probably https://github.com/backuppc/backuppc/pull/79 would help with your issue if you have to use --config-dir .


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Richard Shaw
2017-03-26 15:30:56 UTC
Permalink
Post by Alexander Moisseev
But you have specified --config-dir instead of --config-path .
Probably https://github.com/backuppc/backuppc/pull/79 would help with
your issue if you have to use --config-dir .
Perhaps... I tried just changing the dir to path and got the following:

Is this a new installation or upgrade for BackupPC? If this is
an upgrade please tell me the full path of the existing BackupPC
configuration file (eg: /etc/BackupPC/config.pl). Otherwise, just
hit return.

--> Full path to existing main config.pl [/etc/BackupPC/]
The file '/etc/BackupPC/' is not a regular file.
Need to specify a valid --config-path for upgrade
error: Bad exit status from /var/tmp/rpm-tmp.eRP9qR (%install)

Thanks,
Richard
Alexander Moisseev
2017-03-26 16:00:53 UTC
Permalink
Post by Alexander Moisseev
But you have specified --config-dir instead of --config-path .
Probably https://github.com/backuppc/backuppc/pull/79 <https://github.com/backuppc/backuppc/pull/79> would help with your issue if you have to use --config-dir .
Is this a new installation or upgrade for BackupPC? If this is
an upgrade please tell me the full path of the existing BackupPC
configuration file (eg: /etc/BackupPC/config.pl <http://config.pl>). Otherwise, just
hit return.
--> Full path to existing main config.pl <http://config.pl> [/etc/BackupPC/]
The file '/etc/BackupPC/' is not a regular file.
Need to specify a valid --config-path for upgrade
error: Bad exit status from /var/tmp/rpm-tmp.eRP9qR (%install)
It won't work.
The configure.pl expects a full path including configuration file name as a value of the --config-path option.
You can't specify a directory. But there is a special case: an empty string.
That's why Craig suggested to use "--config-path ''" <-- empty string
It is an equivalent of "hit return" in interactive mode.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-27 19:22:52 UTC
Permalink
Richard,

If I take that command, and add

--config-path ''

then it ignores the current installation and seems to run correctly.

Craig
Post by Richard Shaw
On Sun, Mar 26, 2017 at 12:19 AM, Craig Barratt <
Post by Craig Barratt
In looking a the code, if config-path is specified then any current
config file is ignored.
If you specify dest-dir then all of the output files are put below there.
So if you specify both --config-path '' and --dest-dir, I believe any
existing install should be ignored.
%{__perl} configure.pl \
--batch \
--backuppc-user=backuppc \
--dest-dir %{buildroot} \
--config-dir %{_sysconfdir}/%{name}/ \
--config-override XferMethod=rsync \
--config-override CgiURL=http://localhost/%{name} \
--config-override ClientNameAlias=0 \
--cgi-dir %{_libexecdir}/%{name} \
--data-dir %{_localstatedir}/lib/%{name}/ \
--html-dir %{_datadir}/%{name}/html/ \
--html-dir-url /%{name}/images \
--install-dir %{_datadir}/%{name} \
--log-dir %{_localstatedir}/log/%{name} \
--hostname localhost \
--no-set-perms \
--uid-ignore
Thanks,
Richard
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Richard Shaw
2017-03-27 19:28:38 UTC
Permalink
On Mon, Mar 27, 2017 at 2:22 PM, Craig Barratt <
Post by Alexander Moisseev
Richard,
If I take that command, and add
--config-path ''
then it ignores the current installation and seems to run correctly.
That got it!

Thanks Craig

Richard
Craig Barratt
2017-03-27 22:29:38 UTC
Permalink
Richard,

I just pushed several fixes to configure.pl. Now if you specify
--dest-dir, any existing BackupPC installation should be ignored (ie, no
need to specify --config-path '' any longer).

I also fixed the PingPath and Ping6Path merging, and noticed a couple of
other things that are now fixed.

One comment, when you use --config-override, you need to specify the exact
text replacement, not the value. So instead of these settings:
--config-override XferMethod=rsync \
--config-override CgiURL=http://localhost/%{name} \
--config-override ClientNameAlias=0 \

you need to do this:

--config-override XferMethod=\"rsync\" \
--config-override CgiURL=\"http://localhost/%{name}\" \
--config-override ClientNameAlias=undef \

(undef is the way to make sure ClientNameAlias is not defined).

Craig
Post by Richard Shaw
On Mon, Mar 27, 2017 at 2:22 PM, Craig Barratt <
Post by Alexander Moisseev
Richard,
If I take that command, and add
--config-path ''
then it ignores the current installation and seems to run correctly.
That got it!
Thanks Craig
Richard
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Richard Shaw
2017-03-28 13:32:23 UTC
Permalink
On Mon, Mar 27, 2017 at 5:29 PM, Craig Barratt <
Post by Alexander Moisseev
Richard,
I just pushed several fixes to configure.pl. Now if you specify
--dest-dir, any existing BackupPC installation should be ignored (ie, no
need to specify --config-path '' any longer).
I also fixed the PingPath and Ping6Path merging, and noticed a couple of
other things that are now fixed.
One comment, when you use --config-override, you need to specify the exact
text replacement, not the value.
Yeah, I had hoped to get away without having those, for some reason they
confuse syntax highlighting in VI making reading the spec file difficult
but I've added them back.

I wasn't sure if the commit that fixed this and PingPath would work as a
patch so I downloaded configure.pl from git and I'm copying over the one in
the 4.1.0 source but it's giving me an error... Perhaps I'm missing another
commit that the updates are dependent on?

+ /usr/bin/perl configure.pl --batch --backuppc-user=backuppc --dest-dir
/home/build/rpmbuild/BackupPC/BUILDROOT/BackupPC-4.1.0-1.fc25.x86_64
--config-dir /etc/BackupPC/ --config-override 'CgiURL="
http://localhost/BackupPC"' --config-override ClientNameAlias=undef
--config-override 'NmbLookupPath="/usr/bin/nmblookup"' --config-override
'ParPath="/usr/bin/par2"' --config-override 'PingPath="/usr/bin/ping"'
--config-override 'Ping6Path="/usr/sbin/ping6"' --config-override
'RrdToolPath="/usr/bin/rrdtool"' --config-override
'RsyncBackupPCPath="/usr/bin/rsync_bpc"' --config-override
'RsyncClientPath="/usr/bin/rsync"' --config-override
'SendmailPath="/usr/sbin/sendmail"' --config-override
'SmbClientPath="/usr/bin/smbclient"' --config-override
'SshPath="/usr/bin/ssh"' --config-override 'TarClientPath="/usr/bin/gtar"'
--config-override 'XferMethod="rsync"' --cgi-dir /usr/libexec/BackupPC
--data-dir /var/lib/BackupPC/ --hostname localhost --html-dir
/usr/share/BackupPC/html/ --html-dir-url /BackupPC/images --install-dir
/usr/share/BackupPC --log-dir /var/log/BackupPC --no-set-perms --uid-ignore
You need to run makeDist first to create a tarball release that includes an
updated configure.pl. After you unpack the tarball, run configure.pl from
there.
error: Bad exit status from /var/tmp/rpm-tmp.mrJ5EP (%install)

Thanks,
Richard
Alexander Moisseev
2017-03-28 13:58:06 UTC
Permalink
Post by Richard Shaw
You need to run makeDist first to create a tarball release that includes an
updated configure.pl <http://configure.pl>. After you unpack the tarball, run configure.pl <http://configure.pl> from
there.
It is not possible to apply a source code patch to configure.pl from the distribution tarball directly and not possible to run the configure.pl source.

To install backuppc from HEAD you can clone it using git or download _source code_ tarball.
Then you should make the backuppc distribution by yourself using makeDist. For instance:

./makeDist --nosyntaxCheck --version 4.1.0





------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
BackupPC-***@lists.sourceforge.net
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Craig Barratt
2017-03-28 15:49:07 UTC
Permalink
Richard,

No, as the error message says, you can't run configure.pl directly from
git. You need to run makeDist first, which merges some important settings
(the list of libraries and binaries, as well as the release version and
date) into configure.pl.

Once you test it and confirm it works I'll release 4.1.1.

Also, several of your --config-override settings could be done
with --bin-path instead, but it should be fine either way.

Craig
Post by Richard Shaw
On Mon, Mar 27, 2017 at 5:29 PM, Craig Barratt <
Post by Alexander Moisseev
Richard,
I just pushed several fixes to configure.pl. Now if you specify
--dest-dir, any existing BackupPC installation should be ignored (ie, no
need to specify --config-path '' any longer).
I also fixed the PingPath and Ping6Path merging, and noticed a couple of
other things that are now fixed.
One comment, when you use --config-override, you need to specify the
exact text replacement, not the value.
Yeah, I had hoped to get away without having those, for some reason they
confuse syntax highlighting in VI making reading the spec file difficult
but I've added them back.
I wasn't sure if the commit that fixed this and PingPath would work as a
patch so I downloaded configure.pl from git and I'm copying over the one
in the 4.1.0 source but it's giving me an error... Perhaps I'm missing
another commit that the updates are dependent on?
+ /usr/bin/perl configure.pl --batch --backuppc-user=backuppc --dest-dir
/home/build/rpmbuild/BackupPC/BUILDROOT/BackupPC-4.1.0-1.fc25.x86_64
--config-dir /etc/BackupPC/ --config-override 'CgiURL="http://localhost/
BackupPC"' --config-override ClientNameAlias=undef --config-override
'NmbLookupPath="/usr/bin/nmblookup"' --config-override
'ParPath="/usr/bin/par2"' --config-override 'PingPath="/usr/bin/ping"'
--config-override 'Ping6Path="/usr/sbin/ping6"' --config-override
'RrdToolPath="/usr/bin/rrdtool"' --config-override
'RsyncBackupPCPath="/usr/bin/rsync_bpc"' --config-override
'RsyncClientPath="/usr/bin/rsync"' --config-override
'SendmailPath="/usr/sbin/sendmail"' --config-override
'SmbClientPath="/usr/bin/smbclient"' --config-override
'SshPath="/usr/bin/ssh"' --config-override 'TarClientPath="/usr/bin/gtar"'
--config-override 'XferMethod="rsync"' --cgi-dir /usr/libexec/BackupPC
--data-dir /var/lib/BackupPC/ --hostname localhost --html-dir
/usr/share/BackupPC/html/ --html-dir-url /BackupPC/images --install-dir
/usr/share/BackupPC --log-dir /var/log/BackupPC --no-set-perms --uid-ignore
You need to run makeDist first to create a tarball release that includes an
updated configure.pl. After you unpack the tarball, run configure.pl from
there.
error: Bad exit status from /var/tmp/rpm-tmp.mrJ5EP (%install)
Thanks,
Richard
------------------------------------------------------------
------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
BackupPC-devel mailing list
List: https://lists.sourceforge.net/lists/listinfo/backuppc-devel
Wiki: http://backuppc.wiki.sourceforge.net
Project: http://backuppc.sourceforge.net/
Loading...