Chris Johns commented on a discussion on wscript: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/257#note_113830

 > +            value = no_unicode(cp.get("version", "revision"))
 > +            # The revision is <major>.<minor>[-label]
 > +            # break is up and update the version
 > +            if "." not in value:
 > +                ctx.fatal("Invalid VERSION revision: no dot")
 > +            vs = value.split(".", 1)
 > +            version_major = vs[0]
 > +            vs[0] = vs[1]
 > +            if "." in vs[0]:
 > +                ctx.fatal("Invalid VERSION revision: too many dots")
 > +            if "-" in vs[0]:
 > +                value = vs[0]
 > +                vs = value.split("-", 1)
 > +                version_label = vs[1]
 > +            version_minor = vs[0]
 > +            prefix = "/opt/rtems/" + version_major

The `prefix` is version specific so it changes when the major number changes. 
The code was refactored to parse `VERSION` and set these values when the help 
is displayed as it was incorrectly printing the defaults before.

Also the prefix as it is does not handle Windows cleanly but that is a separate 
issue.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/257#note_113830
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to