Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Use more recent glib suppression list #1179

Merged
merged 3 commits into from
Apr 15, 2019
Merged

Conversation

lbonn
Copy link
Contributor

@lbonn lbonn commented Apr 12, 2019

From flatpack project.

An example of these was found when running the suite under ptest.

From flatpack project.

An example of these was found when running the suite under ptest.

Signed-off-by: Laurent Bonnans <[email protected]>
@codecov-io
Copy link

codecov-io commented Apr 15, 2019

Codecov Report

Merging #1179 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1179   +/-   ##
=======================================
  Coverage   77.97%   77.97%           
=======================================
  Files         170      170           
  Lines       10006    10006           
=======================================
  Hits         7802     7802           
  Misses       2204     2204

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3b10468...3d8447c. Read the comment docs.

@@ -23,5 +23,5 @@ cp -rf "$GITREPO_ROOT/tests/test_data/fake_root" "$TEST_INSTALL_DESTDIR"
cp -rf "$GITREPO_ROOT/src/aktualizr_repo/run/create_repo.sh" "$TEST_INSTALL_DESTDIR"
cp -rf "$GITREPO_ROOT/src/aktualizr_repo/run/serve_repo.py" "$TEST_INSTALL_DESTDIR"

git -C "$GITREPO_ROOT" fetch --tags --depth=10000 || true
git -C "$GITREPO_ROOT" fetch --tags --unshallow || true
Copy link
Collaborator

Choose a reason for hiding this comment

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

Too bad, I was worried that wouldn't work.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, @mike-sul mentioned that it wouldn't work on his docker environment: #1160 (comment). But shouldn't be a problem, since it's suffixed by || true

Not worth the trouble

Signed-off-by: Laurent Bonnans <[email protected]>
@lbonn
Copy link
Contributor Author

lbonn commented Apr 15, 2019

In the end, I disabled shallow clones on Travis. A full clone is relatively fast compared to the docker container builds.

@pattivacek pattivacek merged commit 9c5ef10 into master Apr 15, 2019
@pattivacek pattivacek deleted the fix/update-glib-suppressions branch April 15, 2019 10:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants