Comment 2 for bug 1959444

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to root (master)

Reviewed: https://review.opendev.org/c/starlingx/root/+/826932
Committed: https://opendev.org/starlingx/root/commit/955ec0af14d547523df4998009e87cec1f94e6f9
Submitter: "Zuul (22348)"
Branch: master

commit 955ec0af14d547523df4998009e87cec1f94e6f9
Author: Scott Little <email address hidden>
Date: Fri Jan 28 14:21:52 2022 -0500

    manifest_get_default_revision must deal with / in branch names

    When the manifest default revision is 'r/stx.6.0',
    manifest_get_default_revision was returning 'r'.
    A sed clause intended to catch '/>' was grabbing the wrong
    '/' character.

    Split the sed expression into two seperate expressions.

    Tested with:
       <default revision="r/stx.6.0" remote="stx-staging"/>
       <default revision="stx.6.0" remote="stx-staging"/>
       <default revision="r/stx.6.0"/>
       <default revision="stx.6.0"/>

    Closes-Bug: 1959444
    Change-Id: I3a271f6f89cea94898598c90ed2d2be59ddc60d5
    Signed-off-by: Scott Little <email address hidden>