shymega-ct commented on PR #2060:
URL: https://github.com/apache/buildstream/pull/2060#issuecomment-3301720357

   I agree that it doesn't /really/ fit with the rest of the document. I
   looked through the docs to find an alternative place, but none felt
   like a good place.
   
   I guess we could just keep the section in `BuildElement` and refer to
   that from other documentation?
   
   On 16.09.2025 01:19, Abderrahim Kitouni wrote:
   >@abderrahim commented on this pull request.
   >
   >
   >
   >> @@ -144,6 +144,29 @@ command contain an ``rm`` statement, as such:
   > This would cause later builds of ``runtime-only.bst`` to no longer
   > conflict on the given file.
   >
   >+Changing the location
   >
   >This section doesn't fit with the rest of the document. The document 
discusses an example project, while this section discusses a theoretical 
example that isn't well defined.
   >
   >I'm not sure what's the best way to deal with this. Extend the example 
project so that this can be applied? That could be too complicated for this 
tutorial. Move it to somewhere else in the documentation? I don't know where 
that could be.
   >
   >-- 
   >Reply to this email directly or view it on GitHub:
   >https://github.com/apache/buildstream/pull/2060#pullrequestreview-3228288291
   >You are receiving this because you authored the thread.
   >
   >Message ID: ***@***.***>
   Best regards,
   --
   Dom Rodriguez (he/him)
   Software Engineer
   
   Codethink Ltd
   
   Codethink delivers cutting edge open source design, development and
   integration services.
   
   https://codethink.co.uk
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to