|
|
DescriptionChange //third_party/minigbm to a shared_library
minigbm has exported functions via the PUBLIC #define and is not designed
to be linked statically. These functions should be exported from
libminigbm rather than libozone.
Committed: https://crrev.com/b04d271f548b30f84c28806fdca654964c4b499e
Cr-Commit-Position: refs/heads/master@{#377143}
Patch Set 1 #Patch Set 2 : s/component/shared_library/ #Patch Set 3 : use the proper soname #Patch Set 4 : sigh, fix on old buildbots #Messages
Total messages: 29 (17 generated)
spang@chromium.org changed reviewers: + dnicoara@chromium.org
Description was changed from ========== Change //third_party/minigbm to a component minigbm has expoted functions via the PUBLIC #define. These functions should be exported from libminigbm.so rather than libozone.so. Otherwise, we'd get multiple definitions if any other target depends on the library. ========== to ========== Change //third_party/minigbm to a component minigbm has exported functions via the PUBLIC #define. These functions should be exported from libminigbm.so rather than libozone.so. Otherwise, we'd get multiple definitions if any other target depends on the library. ==========
Description was changed from ========== Change //third_party/minigbm to a component minigbm has exported functions via the PUBLIC #define. These functions should be exported from libminigbm.so rather than libozone.so. Otherwise, we'd get multiple definitions if any other target depends on the library. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm.so.1 rather than chrome. ==========
The CQ bit was checked by spang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1716353002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1716353002/20001
The CQ bit was checked by spang@chromium.org to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1716353002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1716353002/40001
Description was changed from ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm.so.1 rather than chrome. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libgbm.so.1 rather than libozone. ==========
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
lgtm
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1716353002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1716353002/40001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
Description was changed from ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libgbm.so.1 rather than libozone. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm.so rather than libozone. ==========
Description was changed from ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm.so rather than libozone. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm rather than libozone. ==========
The CQ bit was checked by spang@chromium.org
The patchset sent to the CQ was uploaded after l-g-t-m from dnicoara@chromium.org Link to the patchset: https://codereview.chromium.org/1716353002/#ps60001 (title: "sigh, fix on old buildbots")
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1716353002/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1716353002/60001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
The CQ bit was checked by spang@chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1716353002/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1716353002/60001
Message was sent while issue was closed.
Description was changed from ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm rather than libozone. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm rather than libozone. ==========
Message was sent while issue was closed.
Committed patchset #4 (id:60001)
Message was sent while issue was closed.
Description was changed from ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm rather than libozone. ========== to ========== Change //third_party/minigbm to a shared_library minigbm has exported functions via the PUBLIC #define and is not designed to be linked statically. These functions should be exported from libminigbm rather than libozone. Committed: https://crrev.com/b04d271f548b30f84c28806fdca654964c4b499e Cr-Commit-Position: refs/heads/master@{#377143} ==========
Message was sent while issue was closed.
Patchset 4 (id:??) landed as https://crrev.com/b04d271f548b30f84c28806fdca654964c4b499e Cr-Commit-Position: refs/heads/master@{#377143} |