You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The original issue was that if running Buildifier on the generated bazel BUILD files, it would sort deps in the wrong order for multi-lib packages (like openssl or xmlsec). This was fixed in #11560 which added a # do not sort comment to the deps section, so that buildifier would not change the order of deps.
Now the # do not sort comment is no longer added, so buildifier will sort the deps again. (It also looks like the BUILD files might already be "prettified", i.e. have the deps sorted alphabetically when generated by conan). This causes link errors for e.g. openssl and xmlsec, due to their component libraries needing to be in a non-alphabetical order.
The issue re-appeared in conan version 1.62. I believe this is the change that broke it: #14959
Describe the bug
Environment Details
Operating System+version: All
Compiler+version: Not relevant
Conan version: 1.62 and later
Python version: All
Description
This is basically a re-opening of #11559.
The original issue was that if running Buildifier on the generated bazel BUILD files, it would sort deps in the wrong order for multi-lib packages (like openssl or xmlsec). This was fixed in #11560 which added a
# do not sort
comment to the deps section, so that buildifier would not change the order of deps.Now the
# do not sort
comment is no longer added, so buildifier will sort the deps again. (It also looks like the BUILD files might already be "prettified", i.e. have the deps sorted alphabetically when generated by conan). This causes link errors for e.g. openssl and xmlsec, due to their component libraries needing to be in a non-alphabetical order.The issue re-appeared in conan version 1.62. I believe this is the change that broke it: #14959
How to reproduce it
See #11559
The text was updated successfully, but these errors were encountered: