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

Additional Configuration for Files and compare examples #124

Open
DarwinJS opened this issue Nov 25, 2021 · 2 comments · May be fixed by #125
Open

Additional Configuration for Files and compare examples #124

DarwinJS opened this issue Nov 25, 2021 · 2 comments · May be fixed by #125

Comments

@DarwinJS
Copy link

Thanks for this great tool.

I have configured it to gather "Directories" (/etc/ actually) but then during a comparison with another system it says "No extra post files found in"

I want it to collect most of the files in /etc/ and compare the contents of the files - not just the list of files.

I can't see a configuration file examples that does seems to do this?

It would also be great to have examples of regular compares and compare-only - I can contribute these once I get a command that works.

@DarwinJS
Copy link
Author

I used the -v switch and discovered it does seem to be doing file diffs, but not discovering any differences.

So for a first time user who ends up comparing two identical file sets in additional gathered files, the only message is "No extra post files found in

" - so it does not reveal that it actually did file compares at all.

Maybe it should also say "No file comparison differences found." if there was not a single file difference in extra collected files?

@DarwinJS DarwinJS linked a pull request Dec 10, 2021 that will close this issue
@popindavibe
Copy link
Contributor

Hello,
Yes, I guess we could add an extra green line for the extra dir check if no difference was found.

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 a pull request may close this issue.

2 participants