Hi Ross,

I'm currently writing the doc patch for LICENSE_FLAGS_DETAILS.
However, I don't find the current behavior optimal...

On 21.06.23 at 16:22, ross.bur...@arm.com wrote:
From: Ross Burton <ross.bur...@arm.com>

Don't prefix the output of LICENSE_FLAGS_DETAILS with "For further
details, see" so that recipes can put arbitrary text in their license
details instead of being limited to a specific sentence structure.

Signed-off-by: Ross Burton <ross.bur...@arm.com>
---
  meta/classes-global/base.bbclass | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/classes-global/base.bbclass b/meta/classes-global/base.bbclass
index 976a2ddee4b..cbda8d12f09 100644
--- a/meta/classes-global/base.bbclass
+++ b/meta/classes-global/base.bbclass
@@ -520,7 +520,7 @@ python () {
                  message = "Has a restricted license '%s' which is not listed in 
your LICENSE_FLAGS_ACCEPTED." % unmatched
                  details = d.getVarFlag("LICENSE_FLAGS_DETAILS", unmatched)
                  if details:
-                    message += " For further details, see %s." % details
+                    message += details

What about
message += ' ' + details
instead?

Otherwise, you have to add a leading space to LICENSE_FLAGS_DETAILS.
What do you think?

Happy to send a patch if that helps...

Cheers
Michael.
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#187411): 
https://lists.openembedded.org/g/openembedded-core/message/187411
Mute This Topic: https://lists.openembedded.org/mt/99677069/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to