You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildstream.apache.org by GitBox <gi...@apache.org> on 2022/08/02 10:43:22 UTC

[GitHub] [buildstream] jjardon opened a new pull request, #1710: doc/source/main_install.rst: Buildstream depends on lzip

jjardon opened a new pull request, #1710:
URL: https://github.com/apache/buildstream/pull/1710

   This is a dependency of the core tar plugin
   
   See https://github.com/apache/buildstream/pull/1705#issuecomment-1199046601


-- 
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


[GitHub] [buildstream] gtristan commented on pull request #1710: doc/source/main_install.rst: Buildstream depends on lzip

Posted by GitBox <gi...@apache.org>.
gtristan commented on PR #1710:
URL: https://github.com/apache/buildstream/pull/1710#issuecomment-1207909600

   I'm not really concerned about the outcome... whether we explicitly list `lzip` as a dependency or not but...
   
   To clarify:
   
   * BuildStream is perfectly capable of functioning without `lzip` installed, it's not required by any means
   * If your BuildStream project uses a `.tar.lz` file, then the `tar` plugin will require `lzip` (this is an exceedingly rare case)
   * If you try to run `BuildStream` to load an element that uses a `.tar.lz` file, it will exit early informing the user that `lzip` was not found on the host
     * So there is at least a user story for informing the user that they need `lzip` in that edge case
   
   What do we prefer to do ?
   


-- 
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


[GitHub] [buildstream] gtristan merged pull request #1710: doc/source/main_install.rst: Buildstream depends on lzip

Posted by GitBox <gi...@apache.org>.
gtristan merged PR #1710:
URL: https://github.com/apache/buildstream/pull/1710


-- 
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