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

static-delta: Change show to display from/to commits #1823

Closed
wants to merge 2 commits into from

Conversation

cgwalters
Copy link
Member

When writing a delta to a file this may not always be recorded
in the filename, and it's useful data.

This also required teaching show to accept a file path.

Note...for some reason test-deltas.sh breaks when run from
a tty - we get SIGTTIN which implies something is reading from
the tty but it wasn't obvious to me what.

When writing a delta to a file this may not always be recorded
in the filename, and it's useful data.

Ref: https://mail.gnome.org/archives/ostree-list/2019-February/msg00000.html

This also required teaching `show` to accept a file path.

Note...for some reason `test-deltas.sh` breaks when run from
a tty - we get `SIGTTIN` which implies something is reading from
the tty but it wasn't obvious to me what.
@davisRoman
Copy link

davisRoman commented Feb 26, 2019

Hi Colin,

Thank you very much for this patch. I think this will be a useful feature for ostree.

I was able to pull and compile your branch,delta-dump-filename, however the show command didn't quite work for me. My machine doesn't have the main ostree commit repo because that lives on our build server. I figured making an empty repo on my machine would suffice as I'm only interested in running the show command to retrieve the start and end commits from the static-delta.

davis@Precision-7920:/tmp$ mkdir ostree-repo
davis@Precision-7920:/tmp$ ostree --repo=ostree-repo init --mode=bare
davis@Precision-7920:/tmp$ ~/ostree/ostree --repo=ostree-repo static-delta show ./static-88cf8a-96bc7b
error: Invalid rev ./static
davis@Precision-7920:/tmp$

Please let me know what you think.

Thank you,

Davis

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

Looks good to me, just some minor nits.

src/libostree/ostree-repo-static-delta-core.c Outdated Show resolved Hide resolved
src/libostree/ostree-repo-static-delta-core.c Show resolved Hide resolved
@jlebon
Copy link
Member

jlebon commented Mar 11, 2019

@rh-atomic-bot r+ 9ee91ef

@rh-atomic-bot
Copy link

⌛ Testing commit 9ee91ef with merge 8d2e9b8...

@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing 8d2e9b8 to master...

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.

4 participants