Page MenuHomeFreeBSD

Update to 7.0.5
ClosedPublic

Authored by dvl on Sep 12 2014, 1:33 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Oct 21, 12:30 AM
Unknown Object (File)
Sun, Oct 12, 10:27 PM
Unknown Object (File)
Thu, Sep 25, 10:46 PM
Unknown Object (File)
Sep 22 2025, 1:07 AM
Unknown Object (File)
Sep 15 2025, 4:35 PM
Unknown Object (File)
Sep 13 2025, 3:42 PM
Unknown Object (File)
Sep 13 2025, 7:42 AM
Unknown Object (File)
Sep 13 2025, 3:32 AM
Subscribers
None

Details

Reviewers
wg
mat
Summary

commit message will be:

  • Upgrade to 7.0.5
  • do not remove etc/bacula; let bacula-client do that
  • use UID and GID provided by ports/UIDs & ports/GIDs
  • minimize use of owner/group

PR: 193532
Submitted by: Kevin Thompson
Approved by: mat (mentor)

Test Plan

https://redports.org/buildarchive/20140911193900-55363/ sysutils/bacula-client-static (slave port)
https://redports.org/buildarchive/20140911193701-56246/ net-mgmt/nagios-check_bacula (slave port)
https://redports.org/buildarchive/20140911193119-77505/ sysutils/bacula-server (fails to build on 10.0-RELEASE/i386 but I suspect a redports issue)
https://redports.org/buildarchive/20140911193103-81899/ sysutils/bacula-bat (slave port)

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

dvl retitled this revision from to Update to 7.0.5.
dvl updated this object.
dvl edited the test plan for this revision. (Show Details)
dvl added reviewers: mat, wg.
mat edited edge metadata.
mat added inline comments.
sysutils/bacula-server/pkg-plist.client
21

Does the user bacula really needs to be able to write there ?

This revision is now accepted and ready to land.Sep 12 2014, 1:39 PM
dvl edited edge metadata.

Consolidate use of owner:group

sysutils/bacula-bat/pkg-plist
4 ↗(On Diff #1594)

The directory is added by bacula-client.

If not removed, we get:

list of filesystem changes from before and after port installation and deinstallation
usr/local/etc/bacula:
user (0, 910)
gid (0, 5)

https://redports.org/~dvl/20140911154000-6051-243541/bacula-server-7.0.5.log

sysutils/bacula-server/Makefile
80

Update URL anchor

159

Without these changes, we get this:

Variable ECHO is recursive.

See https://redports.org/~dvl/20140911154000-6051-243541/bacula-server-7.0.5.log

sysutils/bacula-server/pkg-plist
78 ↗(On Diff #1594)

Directory created by bacula-client

sysutils/bacula-server/pkg-plist.client
21

Directory created by bacula-client

21

The bacula user does not need to write there, no.

21

FWIW, on my own installation, I have:

$ ls -ld /usr/local/etc/bacula/
drwxr-xr-x 2 root wheel 43 Sep 11 01:41 /usr/local/etc/bacula/

sysutils/bacula-server/pkg-plist
2 ↗(On Diff #1595)

That one should be 640 too, I think.

sysutils/bacula-server/Makefile
194–195

That is wrong. Change the mode in the plist.

With recent changes, and initial early acceptance of the patches by my mentors, I think an explicit 'good to go' is required in the comments before I proceed.

sysutils/bacula-server/pkg-plist
2 ↗(On Diff #1595)

Looking at what installed for me:

$ ls -l /usr/local/etc/bacula/
total 149
-rw-r--r-- 1 root wheel 949 Sep 12 14:39 bacula-barcodes
-rw-r--r-- 1 root wheel 949 Sep 12 14:39 bacula-barcodes.sample
-rw-r----- 1 root bacula 9034 Sep 12 14:39 bacula-dir.conf
-rw-r----- 1 root bacula 9034 Sep 12 14:39 bacula-dir.conf.sample
-rw-r----- 1 root wheel 1031 Sep 12 14:36 bacula-fd.conf
-rw-r----- 1 root wheel 1031 Sep 12 14:36 bacula-fd.conf.sample
-rw-r----- 1 root wheel 7748 Sep 12 14:39 bacula-sd.conf
-rw-r----- 1 root wheel 7748 Sep 12 14:39 bacula-sd.conf.sample
-rw-r----- 1 root bacula 190 Sep 12 14:36 bconsole.conf
-rw-r----- 1 root bacula 190 Sep 12 14:36 bconsole.conf.sample

2 ↗(On Diff #1595)

Which tells me, they all should be done (,bacula,)

Update directory permissions to allow catalog backups to succeed.

An updated was posted to this code review in error.