aboutsummaryrefslogtreecommitdiffstats
path: root/bin/get-pick-list.sh
Commit message (Collapse)AuthorAgeFilesLines
* get-pick-list: Add --pretty=medium to the arguments for Cc patchesDylan Baker2019-02-131-2/+2
| | | | | | | | | | | | | | Because none of them have been picked up for 19.0 due to this bug being reintroduced. v2: - Fix fixes tags Fixes: e6b3a3b2014413366110f6deeced8095e7262b1d ("bin/get-pick-list.sh: handle "typod" usecase.") Fixes: fac10169bbad2da918ef07a62c01e0b321508cfe ("bin/get-pick-list.sh: prefix output with "[stable] "") Reviewed-by: Andres Gomez <[email protected]> Reviewed-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: fix redirection in shAndres Gomez2019-01-141-1/+1
| | | | | | | | | | | | | | "&>" is bash specific. Fixes: e0dbfc99537 ("bin/get-pick-list.sh: warn when commit lists invalid sha") Cc: Juan A. Suarez <[email protected]> Cc: Eric Engestrom <[email protected]> Cc: Dylan Baker <[email protected]> Cc: Emil Velikov <[email protected]> Signed-off-by: Andres Gomez <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> Reviewed-by: Emil Velikov <[email protected]> Reviewed-by: Juan A. Suarez <[email protected]>
* bin/get-pick-list.sh: fix the oneline printingAndres Gomez2019-01-141-1/+1
| | | | | | | | | | | | | | | | | | | "--summary" will also print extended header information such as creations, renames and mode changes. Let's just use "--no-patch", which suppresses the diff output. v2: Use "--no-patch" instead of the "-s" abbreviation (Eric). Fixes: 559c32d2412 ("bin/get-pick-list.sh: simplify git oneline printing") Cc: Juan A. Suarez <[email protected]> Cc: Eric Engestrom <[email protected]> Cc: Dylan Baker <[email protected]> Cc: Emil Velikov <[email protected]> Signed-off-by: Andres Gomez <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> Reviewed-by: Emil Velikov <[email protected]> Reviewed-by: Juan A. Suarez <[email protected]>
* bin/get-pick-list.sh: warn when commit lists invalid shaEmil Velikov2018-12-211-0/+11
| | | | | | | | | | | We had cases where people would list old/invalid sha in the commit. Add a trivial checker to catch those and throw a warning. CC: Juan A. Suarez <[email protected]> CC: Dylan Baker <[email protected]> CC: [email protected] Signed-off-by: Emil Velikov <[email protected]> Reviewed-by: Andres Gomez <[email protected]>
* bin/get-pick-list.sh: rework handing of sha nominationsEmil Velikov2018-12-211-16/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | Currently our is_sha_nomination does: - folds any whitespace, attempting to extract sha-like information - checks that at least one of the shas has landed Split it in two and do sha-like validation first. This way, commits with mesa-stable and sha nominations will feature the fixes/revert/etc instead of stable (a) or will be omitted if not applicable for the respective branch (b). Misc examples from 18.3 (a) -[ stable ] 5bc509363b6 glx: make xf86vidmode mandatory for direct rendering +[ fixes ] 5bc509363b6 glx: make xf86vidmode mandatory for direct rendering (b) -[ stable ] 9a7b3199037 anv/query: flush render target before copying results CC: Juan A. Suarez <[email protected]> CC: Dylan Baker <[email protected]> CC: [email protected] Signed-off-by: Emil Velikov <[email protected]> Reviewed-by: Andres Gomez <[email protected]>
* bin/get-pick-list.sh: handle reverts prior to the branchpointEmil Velikov2018-11-151-1/+8
| | | | | | | | | | | | | | | | | | | Currently we detect when a breaking commit: - has landed in stable, and - is referenced by a untagged fix in master Yet we did not consider the case of breaking commit: - prior to the branchpoint, and - is referenced by a untagged fix in master Addressing the latter is extremely slow, due to the size of the lookup. That said, we can trivially use the existing is_sha_nomination() helper to catch reverts. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: use test instead of [ ]Emil Velikov2018-11-151-4/+4
| | | | | | | | | Latter is rather picky wrt surrounding white space. The explicit `test` doesn't have that problem, plus the statements read a bit easier. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: handle unofficial "broken by" tagEmil Velikov2018-11-151-1/+8
| | | | | | | | | | We have a number of cases were devs will use a tag "broken by". While it's not something officially documented or recommended, checking for it is trivial enough. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: handle fixes tag with missing colonEmil Velikov2018-11-151-1/+5
| | | | | | | | | Every so often, we forget to add the colon after "fixes". Trivially tweak the script to catch it. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: flesh out is_sha_nominationEmil Velikov2018-11-151-2/+9
| | | | | | | | | Refactor is_fixes_nomination into a is_sha_nomination helper. This way we can reuse it for more than the usual "Fixes:" tag. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: tweak the commit sha matching patternEmil Velikov2018-11-151-1/+1
| | | | | | | | | | | | | | | | | | Currently we match on: - any arbitrary length of, - any a-z A-Z and 0-9 characters At the same time, a commit sha consists of lowercase hexadecimal numbers. Any sha shorter than 8 characters is ambiguous - in some cases even 11+ are required. So change the pattern to a-f0-9 and adjust the length to 8-40. As we're here we could use a single grep, instead of the grep/sed combo. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: handle the fixes tagEmil Velikov2018-11-151-3/+43
| | | | | | | | | | | | | Having a separate script to handle the fixes tag, brings a number of issues, so let's fold it in get-pick-list.sh. v2: - pass the sha as argument to the function - Keep original sed pattern Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: handle "typod" usecase.Emil Velikov2018-11-151-1/+8
| | | | | | | | | | | | | | | | As the comment in get-typod-pick-list.sh says, there's little point in having a duplicate file. Add the new pattern + tag to get-pick-list.sh and nuke this file. v2: - pass the sha as argument to the function - grep -q instead of using a variable (Eric) Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: prefix output with "[stable] "Emil Velikov2018-11-151-0/+15
| | | | | | | | | | | | | | | | | | | | With later commits we'll fold all the different scripts into one. Add the explicit prefix, so that we know the origin of the nomination v2: - pass the sha as argument to the function - swap $tag = none for an else statment (Juan) - grep -q instead of using a variable (Eric) - print the tag and commit oneline separately (Eric) v3: - drop unused "tag=none" assignment (Juan) - typo nomination Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> (v2) Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-list.sh: simplify git oneline printingEmil Velikov2018-11-151-1/+1
| | | | | | | | | | | | Currently we force disable the pager via "|cat" where --no-pager exists. Additionally we could use git show instead of git log -n1. Use those for a slightly more understandable code. Cc: [email protected] Reviewed-by: Juan A. Suarez <[email protected]> Reviewed-by: Eric Engestrom <[email protected]> Signed-off-by: Emil Velikov <[email protected]>
* bin/get-pick-listh.sh: force git --pretty=mediumDylan Baker2018-05-231-1/+1
| | | | | | Signed-off-by: Dylan Baker <[email protected]> Reviewed-by: Kenneth Graunke <[email protected]> Reviewed-by: Andres Gomez <[email protected]>
* bin/get-pick-list.sh: remove ancient way of nominating patchesEmil Velikov2017-02-161-1/+1
| | | | | | | | | The old way of nominating patches [NOTE: .*[Cc]andidate] was deprecated and has been unused for approx. 3 years. Cc: "13.0 17.0" <[email protected]> Signed-off-by: Emil Velikov <[email protected]> Reviewed-by: Eric Engestrom <[email protected]>
* bin/get-pick-list.sh: limit `git grep ...' only as neededEmil Velikov2017-02-161-2/+5
| | | | | | | | Analogous to previous commit. Cc: "13.0 17.0" <[email protected]> Signed-off-by: Emil Velikov <[email protected]> Reviewed-by: Eric Engestrom <[email protected]>
* get-pick-list: Allow for non-whitespace between "CC:" and "mesa-stable"Carl Worth2013-07-311-1/+1
| | | | | | | | | | We recently proposed a new syntax for stable-patch nominations such as: CC: "9.2 and 9.1" <[email protected]> and this has already appeared in the wild. So we extend the regular expression to pick this up as well.
* get-pick-list.sh: Include commits mentionining "CC: mesa-stable..." in pick listCarl Worth2013-07-301-1/+1
| | | | | | | | | | | | | | We recently adopted a new convention that patches can be nominated for the stable branch by including a line in the commit message as follows: CC: [email protected] This is a convenient syntax as "git send-email" will notice this line and automatically copy the resulting patch email to the mesa-stable mailing list. Here we extend the regular expression in the get-pick-list.sh script to also notice this pattern, (as well as the traditional "NOTE: This patch is a candidate..." form.
* mesa: add usage examples to get-pick-list and shortlog scriptsAndreas Boll2013-05-011-0/+6
| | | | NOTE: This is a candidate for the stable branches.
* mesa: Modify candidate search stringIan Romanick2013-03-051-1/+1
| | | | | | | | | Several commits on master for the 9.1 branch had "NOTE" messages in a slightly different format. NOTE: This is a candidate for stable branches Signed-off-by: Ian Romanick <[email protected]>
* mesa: use .cherry-ignore in the get-pick-list.sh scriptAndreas Boll2012-11-141-2/+2
| | | | | | NOTE: This is a candidate for the stable branches. Reviewed-by: Ian Romanick <[email protected]>
* mesa: fix indentation in get-pick-list.sh scriptAndreas Boll2012-10-231-4/+4
| | | | | | NOTE: This is a candidate for the stable branches. Reviewed-by: Ian Romanick <[email protected]>
* mesa: grep for commits with cherry picked in commit message only onceAndreas Boll2012-10-231-2/+9
| | | | | | | | and save them temporary in already_picked NOTE: This is a candidate for the stable branches. Reviewed-by: Ian Romanick <[email protected]>
* mesa: optimize get-pick-list.sh scriptAndreas Boll2012-10-231-4/+3
| | | | | | | | | cuts down the while loop iterations from 4600 to 380 commits at the moment NOTE: This is a candidate for the stable branches. Reviewed-by: Ian Romanick <[email protected]>
* mesa: simplify get-pick-list.sh scriptAndreas Boll2012-10-231-1/+3
| | | | | | | | and add a description for the script NOTE: This is a candidate for the stable branches. Reviewed-by: Ian Romanick <[email protected]>
* mesa: add get-pick-list.sh script into bin/Ian Romanick2012-10-231-0/+21
NOTE: This is a candidate for the stable branches.
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-02 21:49:15 +0000