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

[qt] Clone the latest combination of qtbase and qtsvg which passed upstream CI #7017

Merged
merged 5 commits into from
Dec 20, 2021

Conversation

rlohning
Copy link
Contributor

The super-repo qt5.git contains the revisions which were last tested successfully with all
submodules together. That is much older.

The super repo can be removed as soon as the build script was updated in upstream repo.

qtsvg/depencies.yaml contains the latest revision of qtbase which qtsvg
was successfully tested with. This is usually much newer than the
submodules in qt5.
@rlohning
Copy link
Contributor Author

afl-clang-fast++ crashing again. Is anybody already working on that? Should be unrelated to this change.

Hoping that recent afl revert fixes the build.
@rlohning
Copy link
Contributor Author

Building Qt passed in #7028 , which did not build Qt Widgets. In case that made the difference, I'll try another run of this here when https://codereview.qt-project.org/c/qt/qtqa/+/386883 was merged upstream, the change which was cherry-picked into #7028.

@rlohning
Copy link
Contributor Author

Could someone please merge this? Because of problems on Qt's (our) side, the tested code is more than a month old already. This change here allow us to circumvent that.

Should anything be missing in the change, please let me know and I'll fix it.

@DavidKorczynski DavidKorczynski merged commit 21189b0 into google:master Dec 20, 2021
@rlohning
Copy link
Contributor Author

Oh, that was quick! Thank you very much. :-)

@rlohning rlohning deleted the use_dependencies branch December 20, 2021 18:47
@DavidKorczynski
Copy link
Collaborator

Oh, that was quick! Thank you very much. :-)

You're welcome :)!

MartinPetkov pushed a commit to MartinPetkov/oss-fuzz that referenced this pull request Aug 15, 2022
…stream CI (google#7017)

* [qt] Clone sources according to qtsvg's latest dependencies

qtsvg/depencies.yaml contains the latest revision of qtbase which qtsvg
was successfully tested with. This is usually much newer than the
submodules in qt5.

* [qt] Still keep the old source checkout

* Empty commit to trigger github actions
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.

2 participants