[I] Declaration order of dependencies affects staging order and cache keys [buildstream]

["juergbi (via GitHub)" <gi...@apache.org>]

juergbi opened a new issue, #1995:
URL: https://github.com/apache/buildstream/issues/1995

   The intent of sorting dependencies (done in `loadelement.pyx`) was that the staging order would be independent of the declaration order of dependencies in `.bst` files. As part of the investigation into #1988, it has become clear that the comparison function is broken as it does not impose a total ordering. Besides resulting in different sort orders with different Python versions (addressed in #1993), this also means that reordering dependencies in `.bst` files sometimes results in a change in staging order, which in turn results in a change in cache key.
   
   BuildStream should implement a better sorting algorithm that is independent of the declaration order. As this would change staging order and cache keys of existing projects, this should be opt-in per project.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@buildstream.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Declaration order of dependencies affects staging order and cache keys [buildstream]

["abderrahim (via GitHub)" <gi...@apache.org>]

abderrahim commented on issue #1995:
URL: https://github.com/apache/buildstream/issues/1995#issuecomment-2663337789

   A new cache key algorithm should also look into fixing https://github.com/apache/buildstream/issues/1893 at the same time.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@buildstream.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org