Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

app-backup/btrbk: add versions 0.23.2 and 9999 #1877

Closed
wants to merge 1 commit into from

Conversation

candrews
Copy link
Member

btrbk is a backup tool for btrfs subvolumes, taking advantage of btrfs specific capabilities to create atomic snapshots and transfer them incrementally to backup locations via ssh.

https://bugs.gentoo.org/show_bug.cgi?id=564780

@candrews
Copy link
Member Author

@gentoo/proxy-maint How's this look?

@gktrk gktrk added the new package The PR is adding a new package. label Jul 13, 2016

DESCRIPTION="Tool for creating snapshots and remote backups of btrfs subvolumes"
HOMEPAGE="https://digint.ch/btrbk/"
LICENSE="GPL-3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The correct license is GPL-3+

@gktrk
Copy link
Member

gktrk commented Jul 13, 2016

@gentoo/proxy-maint

@gktrk
Copy link
Member

gktrk commented Jul 13, 2016

Is there a reason why we prefer manual installation over something like:
emake DESTDIR="${D}" DOCDIR="${D}/usr/share/${PF}" install

@candrews candrews force-pushed the bug564780 branch 2 times, most recently from d42a24a to 048181b Compare July 13, 2016 16:20
@candrews
Copy link
Member Author

@gktrk good point, this change really reduces the number of lines / complexity of the ebuild and improves maintainability.
Here's what I'm doing now:

src_install() {
        emake DESTDIR="${D}" DOCDIR="/usr/share/doc/${PF}" install
}

I compared the output of equery files btrbk before and after this change. They're identical except for 2 small differences:

  • The ChangeLog isn't installed by the Makefile - I don't think this is worth doing in the ebuild, but I asked upstream to make this change: Include ChangeLog in install-doc digint/btrbk#95
  • The ebuild was installing the scripts in /usr/share/btrbk instead of /usr/share/btrbk/scripts (which the Makefile does). Doing what upstream does is the way to go.

@candrews
Copy link
Member Author

@digint do you want to be included as a maintainer in metadata.xml?

@digint
Copy link
Contributor

digint commented Jul 13, 2016

Yes you can add me as a maintainer.

Regarding the Makefile: are the systemd unit files copied to the correct place when using emake install rather than systemd_dounit ? If so, I agree that it's much cleaner this way.

You might also consider depending on sys-apps/pv. This is only used for the --progress option and rate_limit config option, but some people (on the debian package) complained about this missing dependency.

@candrews I'm releasing btrbk-0.23.3 later tonight (small bugfix, also adding missing ChangeLog), yell if there's something missing.

@candrews candrews force-pushed the bug564780 branch 2 times, most recently from 75bc948 to a950697 Compare July 14, 2016 01:01
@candrews
Copy link
Member Author

@digint,

Yes you can add me as a maintainer.

Excellent, I've updated metadata.xml accordingly.

Regarding the Makefile: are the systemd unit files copied to the correct place when using emake install rather than systemd_dounit? If so, I agree that it's much cleaner this way.

In both cases, the files are copied to /usr/lib/systemd/system which is probably almost always right. Thanks for point this out - I've changed the ebuilds to pass SYSTEMDDIR="$(systemd_get_systemunitdir)" to emake so that the path is always right :-)

You might also consider depending on sys-apps/pv. This is only used for the --progress option and rate_limit config option, but some people (on the debian package) complained about this missing dependency.

I added a use flag to toggle this dependency - I think that approach makes sense.

I'm releasing btrbk-0.23.3 later tonight (small bugfix, also adding missing ChangeLog), yell if there's something missing.

Thanks!

btrbk is a backup tool for btrfs subvolumes, taking advantage of btrfs specific capabilities to create atomic snapshots and transfer them incrementally to backup locations via ssh.

Gentoo-bug: 564780
@digint
Copy link
Contributor

digint commented Jul 14, 2016

@candrews thanks, I can confirm it working at my place.

@gktrk
Copy link
Member

gktrk commented Jul 14, 2016

Merge in 932c16f. Thanks!

I've made the following changes before merge:

  • Removed extra empty line at the end of metadata.xml
  • Mangled the commit message summary line slightly

@gktrk gktrk closed this Jul 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new package The PR is adding a new package.
Projects
None yet
3 participants