You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@buildstream.apache.org by GitBox <gi...@apache.org> on 2021/01/16 07:43:57 UTC

[GitHub] [buildstream] gtristan opened a new pull request #1442: Tristan/optional project

gtristan opened a new pull request #1442:
URL: https://github.com/apache/buildstream/pull/1442


   This merge requests removes the requirement that BuildStream should be launched within a project directory, or that `-C/--directory` should be specified to indicate a project directory.
   
   When `Stream` finds itself trying to load an `Element`, then an Error is raised if a `Project` was not loaded, but not if only `ArtifactElement`s are loaded.
   
   Tests for the various commands which support operation on artifact names have been enhanced to also test the cases where a project is not loaded.
   


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

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



[GitHub] [buildstream] cs-shadow commented on pull request #1442: Tristan/optional project

Posted by GitBox <gi...@apache.org>.
cs-shadow commented on pull request #1442:
URL: https://github.com/apache/buildstream/pull/1442#issuecomment-763200423


   I understand this PR has other dependencies but just wanted to give it an overall +1. It'll be handy to not have to specify a project in such cases.


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

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



[GitHub] [buildstream] gtristan commented on pull request #1442: Tristan/optional project

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


   This pull requests depends on first landing https://github.com/apache/buildstream/pull/1433


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

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



[GitHub] [buildstream] gtristan merged pull request #1442: Stop requiring project.conf for all commands

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


   


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

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