Cool. One more resolved.
There has been some confusion on what happens next once we get to this state.
Do we:
1)
Mark this patch as abandoned in the spreadsheet and wait to remove it until we rebase?
2)
Mark the patch as abandoned and remove it from the code base?
3)
Mark the patch as deleted and remove it from the code base now?
4)
Something else?
brucej
Hello Huifeng,
You are correct. This commit should not be upstreamed since the underlying features are being upstreamed, refactored or abandoned.
Regards, Matt
From:
"Le, Huifeng" <huifeng.le@intel.com>
Date: Tuesday, November 20, 2018 at 3:54 AM
To: "Peters, Matt" <Matt.Peters@windriver.com>
Cc: "starlingx-discuss@lists.starlingx.io" <starlingx-discuss@lists.starlingx.io>
Subject: Questions about patch ab1baac upstreaming
Matt,
I am looking at patch#ab1baac (US106501: Change to Titanium’s handling Neutron Extensions) which targets to remove wrs-fields from the response to non-wrs client (e.g. no ‘wrs-header’ in the client
request), suppose this is STX special feature and not need for upstream, could you please help to confirm?
Thanks much!
Best Regards,
Le, Huifeng