| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
Per discussion on IRC, really all we need in the summary is the ref ->
commit data. We can easily fetch the commits in a separate phase, and
avoiding duplication is good.
Also, to allow future extensibility, add in a generic a{sv}.
|
| |
|
|
|
|
|
|
|
| |
Otherwise, we're potentially holding up subsequent requests.
I was hitting this when testing the metalink code, where we want to
continue doing more fetches after hitting a 404.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
I'd like to use GTask, so bump the dependency.
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
We want to write to the dest repo, not src.
Noticed while reviewing this code for some other bug.
https://bugzilla.gnome.org/show_bug.cgi?id=733579
|
|
|
|
|
|
|
|
|
|
|
|
| |
The user might "ostree ls /usr/bin/bash/blah", which previously would
segfault.
A somewhat related future enhancement here would be for "ostree ls" to
follow symbolic links.
Reported-by: Dusty Mabe <dustymabe@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=733476
|
|
|
|
|
|
| |
An embarassing off-by-one here. I noticed we weren't pruning them.
https://bugzilla.gnome.org/show_bug.cgi?id=733458
|
|
|
|
|
|
|
|
|
|
| |
Prune has worked fine on bare repositories for some time, but now that
I finally try to delete data on the server side, I notice we weren't
actually enumerating content objects =/
That caused them to not be pruned.
https://bugzilla.gnome.org/show_bug.cgi?id=733458
|
|
|
|
|
|
|
|
|
|
| |
The prune API duplicated logic to delete objects, and furthermore the
core API to delete an object didn't clean up detached metadata.
Fix the duplication by doing the obvious thing: prune should call
_delete.
https://bugzilla.gnome.org/show_bug.cgi?id=733452
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch adds a function that will parse a partial checksum when
resolving a refspec. If the inputted refspec matches a truncated
existing checksum, it will return that checksum to be parsed. If
multiple truncated checksums match the partial refspec, it is not
unique and will return false. This addition is inspired by the same
functionality in Docker, which allows a user to reference a specific
commit without typing the entire checksum.
partial checksums: Add function to abstract comparison
This modifies the list_objects and list_objects_at functions
to take an additional argument for the string that a commit starts
with. If this string arg is not null, it will only list commit
objects beginning with that string. This allows for a new function
ostree_repo_list_commit_objects_starting_with to pass a partial string
and return a list of all matching commits. This improves on the
previous strategy of listing refs because it will list all commit objects,
even ones in past history. This update also includes bugfixes on
error handling and string comparison, and changes the output structure
of resolve_partial_checksum. The new strcuture will no longer return FALSE
without error. Also, the hashtable foreach now uses iter. Also
includes modified test file
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
/var needs to be read-write for a functioning system. Various
systemd services will fail to start if /var is read-only. After we
remount /var (or if we skip remounting /var because / is read-only),
if /var is still readonly, mount a tmpfs file system on /var.
While this isn't strictly part of ostree, keeping it here makes sense
because it keeps twiddling around with the /var mount in one place
for easier coordination and maintenance. This will likely need updating
if systemd gains better support for a readonly root filesystem.
https://bugzilla.gnome.org/show_bug.cgi?id=732979
|
|
|
|
|
|
|
| |
Otherwise, GTlsFileDatabase ends up accepting NULL for anchors, and
then we don't trust any CAs at all.
https://bugzilla.gnome.org/show_bug.cgi?id=726256
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Some organizations will want to use private Certificate Authorities to
serve content to their clients. While it's possible to add the CA
to the system-wide CA store, that has two drawbacks:
1) Compromise of that cert means it can be used for other web traffic
2) All of ca-certificates is trusted
This patch allows a much stronger scenario where *only* the CAs in
tls-ca-path are used for verification from the given repository.
https://bugzilla.gnome.org/show_bug.cgi?id=726256
|
|
|
|
|
| |
The goal of this is to hard-fail if the API isn't present in the
buildroot, as some downstreams may explicitly require it.
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Using docker as a model, this update creates separate manpages
for each OSTree command, such that the main manpage is not
cluttered and the separate pages can provide more in-depth detail
and exanples that might be useful to a user. Each page includes
synopsis, description, example, and a list of options if needd.
This update also alphabetizes the usage error output for ostree
and ostree admin so that it matches the list on the manpage.
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
|
|
|
| |
We need to end the status line *after* we've done a pull, as ostree
admin upgrade does. Also add the correct in_status_line assignment.
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
|
|
|
| |
One was an unused variable, the other is actually dead because we
can't have mfile != NULL.
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
|
|
|
|
|
|
|
| |
We were using unsigned size when we should have been using signed,
this means we basically weren't checking for errors on write...ouch.
Luckily if we e.g. hit ENOSPC during a pull, the checksums wouldn't
match and we'd return an error anyways. However when writing an
object, we'd end up silently ignoring it =/
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
|
|
| |
This condition can't actually be hit, let's hint that's the case.
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=732020
|
| |
|
|
|
|
|
|
| |
Only access relevant portion of argv after we've verified argc.
https://bugzilla.gnome.org/show_bug.cgi?id=731984
|
|
|
|
|
|
| |
branches is always NULL at that point, we want to look at argc.
https://bugzilla.gnome.org/show_bug.cgi?id=731984
|
|
|
|
|
|
|
|
| |
At least one external tool is using the API, and wants to add a
remote, but all of the logic right now is in the tool. Move it to the
library.
https://bugzilla.gnome.org/show_bug.cgi?id=731984
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=731924
|
|
|
|
|
| |
Link to docker, note in introductory paragraph the goal of package
composition on a server.
|
| |
|
| |
|
| |
|
|
|
|
| |
To be more clear that we don't handle "inventory".
|
|
|
|
|
|
|
|
|
|
| |
The generic GKeyFile error isn't quite informative enough here.
I hit this with the new compose process where we don't automatically
inject a configured remote into the generated disk images; we expect
people to add them.
https://bugzilla.gnome.org/show_bug.cgi?id=731346
|
| |
|
| |
|
|
|
|
|
| |
Per discussion on mailing list, the --disable-fsync has a
double-negative thing going on; --fsync=false is clearer.
|
| |
|