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

lib/repo: Add ostree_repo_hash() and tests #1205

Closed
wants to merge 1 commit into from

Conversation

pwithnall
Copy link
Member

@pwithnall pwithnall commented Sep 21, 2017

Add a hash function for OstreeRepo instances, which relies on the repo
being open, and hence being able to hash the device and inode of its
root directory.

Add unit tests for this and ostree_repo_equal().

Signed-off-by: Philip Withnall [email protected]

#1191

@pwithnall pwithnall self-assigned this Sep 21, 2017
Add a hash function for OstreeRepo instances, which relies on the repo
being open, and hence being able to hash the device and inode of its
root directory.

Add unit tests for this and ostree_repo_equal().

Signed-off-by: Philip Withnall <[email protected]>

ostreedev#1191
@cgwalters
Copy link
Member

Nice, thanks!

@rh-atomic-bot r+ eb2f454

{
g_autoptr(GError) error = NULL;

(void) glnx_mkdtemp ("test-repo-XXXXXX", 0700, &fixture->tmpdir, &error);
Copy link
Member

Choose a reason for hiding this comment

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

Side note; ostree's test infra automatically allocates a tmpdir (and so does gnome-desktop-testing-runner). But this is OK, it does make it easier to run just one test (which is a whole issue in itself).

@rh-atomic-bot
Copy link

⌛ Testing commit eb2f454 with merge 64b23fd...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: cgwalters
Pushing 64b23fd to master...

@pwithnall pwithnall deleted the 1191-repo-hash branch September 22, 2017 10:57
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