RFR: 934: Bots incorrectly assigned "build" label despite no obvious matching rule

Robin Westberg rwestberg at openjdk.java.net
Thu Apr 15 07:19:15 UTC 2021


On Wed, 14 Apr 2021 15:24:22 GMT, Robin Westberg <rwestberg at openjdk.org> wrote:

> When determining the set of file names to evaluate for determining label assignment, only include the source name for deletions and renames (and not for copies, for example).

Yes, that was indeed the case! Perhaps I should mention that it was actually @edvbld who figured it out.. :) So the diff we looked at was (dropped a few similar entries):


> git diff 15bcf6d9 0bce6a53 --find-renames=90% --find-copies=90% --find-copies-harder --name-status
M       src/jdk.jpackage/linux/classes/jdk/jpackage/internal/LinuxAppImageBuilder.java
R100    src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/java32.png       src/jdk.jpackage/linux/classes/jdk/jpackage/internal/resources/JavaApp.png
M       src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/MacAppBundler.javarces_ja.properties
M       src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/resources/MacResources_zh_CN.properties
A       src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/resources/product-def.plist
C091    make/data/macosxsigning/java.plist      src/jdk.jpackage/macosx/classes/jdk/jpackage/internal/resources/sandbox.plist
M       src/jdk.jpackage/share/classes/jdk/jpackage/internal/Arguments.java
M       src/jdk.jpackage/windows/classes/jdk/jpackage/internal/WindowsAppImageBuilder.java
R100    src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resources/java48.ico     src/jdk.jpackage/windows/classes/jdk/jpackage/internal/resources/JavaApp.ico
M       test/jdk/tools/jpackage/helpers/jdk/jpackage/test/LauncherIconVerifier.java


So there was indeed a "copy" that seemed to come from the make folder..

-------------

PR: https://git.openjdk.java.net/skara/pull/1118


More information about the skara-dev mailing list