search

toradex / meta-toradex-security

MIT License
4 stars 9 forks source link

u-boot: display warnings on disallowed commands #59

Closed rborn-tx closed 2 months ago

rborn-tx commented 2 months ago

Same as https://github.com/toradex/meta-toradex-security/pull/57 but for the main branch.

sergioprado commented 2 months ago

@rborn-tx we haven't discussed this yet. But I think we should not maintain the main branch (there are no users of the main branch, no automated builds, etc). I think it is too much overhead with little benefits for now.

We can maintain just kirkstone and scarthgap, and leave the main branch as is. As soon as we decide to work on a new OE version, then we can cherry-pick from scarthgap to main and fork a new version, just as we have done with scarthgap.

Does it make sense? Or you see any advantages on maintaining the main branch as well?

rborn-tx commented 2 months ago

@rborn-tx we haven't discussed this yet. But I think we should not maintain the main branch (there are no users of the main branch, no automated builds, etc). I think it is too much overhead with little benefits for now.

We can maintain just kirkstone and scarthgap, and leave the main branch as is. As soon as we decide to work on a new OE version, then we can cherry-pick from scarthgap to main and fork a new version, just as we have done with scarthgap.

Does it make sense? Or you see any advantages on maintaining the main branch as well?

@sergioprado That seems perfectly sensible to me. We can even drop this PR and sync main in batches when needed.

P.S.: That reminds me that we should switch our manifests to using the scarthgap-7.x.y branch of meta-toradex-security rather than master; I'll go ahead and o that.

sergioprado commented 2 months ago

@sergioprado That seems perfectly sensible to me. We can even drop this PR and sync main in batches when needed.

Perfect. I will close this PR then.

P.S.: That reminds me that we should switch our manifests to using the scarthgap-7.x.y branch of meta-toradex-security rather than master; I'll go ahead and o that.

Yeah, that is the perfect time to do this. :-)