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

[DOC] #4118 - Explain difference between --exclude and --pattern. #4783

Conversation

fantasya-pbem
Copy link
Contributor

In my opinion the borg help pattern is already quite good in explaining the --exclude and --pattern features, so I just added a note that explicitly states the additional prefix in --pattern lines.
If you read this help carefully it should be sufficient.

@codecov-io
Copy link

codecov-io commented Oct 13, 2019

Codecov Report

Merging #4783 into master will increase coverage by 0.07%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4783      +/-   ##
==========================================
+ Coverage   83.84%   83.91%   +0.07%     
==========================================
  Files          37       37              
  Lines        9762     9762              
  Branches     1622     1622              
==========================================
+ Hits         8185     8192       +7     
+ Misses       1103     1097       -6     
+ Partials      474      473       -1
Impacted Files Coverage Δ
src/borg/archiver.py 81.25% <ø> (ø) ⬆️
src/borg/archive.py 82.98% <0%> (+0.48%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 73ab1f1...13b2675. Read the comment docs.


Via ``--pattern`` or ``--patterns-from`` you can define BOTH inclusion and exclusion
of files using pattern prefixes ``+`` and ``-``. With ``--exclude`` and
``--exlude-file`` ONLY excludes are defined.
Copy link
Member

Choose a reason for hiding this comment

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

it is not --exlude-file, but --exclude-from.

one could also continue the last sentence with "... and you need to use --include ... to define includes ONLY.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Stupid me...

But I'm not sure what you mean with --include. There is no --include mentioned in e.g. borg create, is this a new undocumented option?

Copy link
Member

@ThomasWaldmann ThomasWaldmann left a comment

Choose a reason for hiding this comment

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

some feedback

src/borg/archiver.py Outdated Show resolved Hide resolved
@ThomasWaldmann ThomasWaldmann merged commit 212b438 into borgbackup:master Oct 19, 2019
@ThomasWaldmann
Copy link
Member

Do we need same change for 1.1-maint?h

@fantasya-pbem fantasya-pbem deleted the doc/4118_Explain-difference-pattern-exclude branch October 20, 2019 07:30
elho pushed a commit to elho/borg that referenced this pull request Mar 17, 2020
…ttern. (borgbackup#4783)

docs: Explain difference between --exclude and --pattern, fixes borgbackup#4118
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants