Ryan Gonzalez (fec4bfdc) at 20 Mar 21:34
dh_setup_copyright: Track sources for files copied to the install p...
Ryan Gonzalez (2c4b3820) at 20 Mar 20:10
Ryan Gonzalez (2c4b3820) at 20 Mar 20:10
Remove legacy-specific host options
The need for this split has been gone since https://phabricator.apertis.org/T9905 was completed, and it's causing problems now since it only supports port 22. Instead, we can merge the options together and use the ssh:// syntax for both ostree-push variants.
https://phabricator.apertis.org/T10209
Signed-off-by: Ryan Gonzalez ryan.gonzalez@collabora.com
Thinking about it more, yeah this probably does need support for progress in the other cases of the function:
total_deltas > 0
: all our update tests actually use static deltas, and they're quite common in OSTree.scanning || outstanding_metadata_fetches
this means that the updater doesn't actually know what the progress it, but it should probably be communicated to the client anyway, e.g. by sending a progress of 0 or -1 or similar.I mean more that the -data
/ Data
suffix is redundant in both places; it doesn't add anything to the property name.
Ah so it's "internal" but internal as in, "this is not the version of the property that's exposed over d-bus"? I do think that's a bit of a confusing name then; if the two properties end up with the same values, there isn't really any reason to distinguish their names.
I don't see why you would need a return value? A wrapper could just load the subset of the progress data needed to decide whether or not to emit an update, e.g. something like (not tested!)
ostree_async_progress_get (ostree_progress,
"outstanding-fetches", "u", &outstanding_fetches,
"outstanding-metadata-fetches", "u", &outstanding_metadata_fetches,
"outstanding-writes", "u", &outstanding_writes,
"caught-error", "b", &caught_error,
"scanning", "u", &scanning,
"status", "s", &status,
NULL);
if (*status == '\0' && !caught_error && outstanding_fetches && !(scanning || outstanding_metadata_fetches))
// emit progress...
(As an aside, I think this should probably calculate percentages for delta updates too?)
In addition to finding the licensing information for each package's binaries, this will now also process the newly added copied_sources metadata to find the copyright of files directly copied from the package sources.
https://phabricator.apertis.org/T7616
Signed-off-by: Ryan Gonzalez ryan.gonzalez@collabora.com
Draft because it depends on pkg/debhelper!29.
In addition to scanning binary files, now we also try to match files directly copied to the install prefix to their original location, which will allow the BOM generation scripts in the images to track their copyright properly.
https://phabricator.apertis.org/T7616
Signed-off-by: Ryan Gonzalez ryan.gonzalez@collabora.com
Ryan Gonzalez (d99ffbbb) at 20 Mar 17:30
generate_licensing_bom: Support the new copied_sources metadata
... and 7 more commits
Ryan Gonzalez (192ef820) at 20 Mar 17:30
dh_setup_copyright: Track sources for files copied to the install p...
... and 4 more commits
Ryan Gonzalez (afed048b) at 20 Mar 17:20
generate_licensing_bom: Support the new copied_sources metadata
... and 1 more commit
Ryan Gonzalez (68bb77ca) at 20 Mar 17:20
dh_setup_copyright: Track sources for files copied to the install p...
Ryan Gonzalez (cea8c440) at 20 Mar 17:17
Ryan Gonzalez (cea8c440) at 20 Mar 17:11
Add gitlab CI setup to build the docker images
Ryan Gonzalez (6e689a31) at 20 Mar 17:05
Add gitlab CI setup to build the docker images
... and 1362 more commits
The version of ostree-push in v2023 does not support the -o option. The port is provided by the crafted 'ssh' command variable instead.
Signed-off-by: Ritesh Raj Sarraf ritesh.sarraf@collabora.com
Ryan Gonzalez (0823de5f) at 18 Mar 19:47
Fix ostree-push options