Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: P4
    • Resolution: Other
    • Affects Version/s: 10
    • Fix Version/s: 10
    • Component/s: infrastructure
    • Labels:
      None

      Description

      The help move fixes across the exploded repos / consolidated repos boundary, it would be helpful to have a script which translated paths in the patches.

      Such a tool would be analogous to the script developed to change paths for the Jigsaw modularity moves and earlier the script to normalized whitespace.

        Activity

        darcy Joe Darcy created issue -
        erikj Erik Joelsson made changes -
        Field Original Value New Value
        Affects Version/s 10 [ 16302 ]
        erikj Erik Joelsson made changes -
        Fix Version/s 10 [ 16302 ]
        erikj Erik Joelsson made changes -
        Status New [ 10000 ] Open [ 1 ]
        Hide
        darcy Joe Darcy added a comment - - edited
        Specifically the modularization patch shuffler is in

        $ROOT/common/bin/unshuffle_patch.sh
        Show
        darcy Joe Darcy added a comment - - edited Specifically the modularization patch shuffler is in $ROOT/common/bin/unshuffle_patch.sh
        darcy Joe Darcy made changes -
        Assignee Chris Hegarty [ chegar ]
        darcy Joe Darcy made changes -
        Parent JDK-8165279 [ 4899485 ]
        Issue Type Sub-task [ 5 ] Bug [ 1 ]
        darcy Joe Darcy made changes -
        Parent JDK-8165178 [ 4899299 ]
        Issue Type Bug [ 1 ] Sub-task [ 5 ]
        Hide
        erikj Erik Joelsson added a comment -
        Attaching a mapping of the file moves that could be used as basis for unshuffle script. The mappings are using full forest paths. This is just for OpenJDK.
        Show
        erikj Erik Joelsson added a comment - Attaching a mapping of the file moves that could be used as basis for unshuffle script. The mappings are using full forest paths. This is just for OpenJDK.
        erikj Erik Joelsson made changes -
        Attachment unshuffle_list.txt [ 72148 ]
        erikj Erik Joelsson made changes -
        Assignee Chris Hegarty [ chegar ] Erik Joelsson [ erikj ]
        Hide
        erikj Erik Joelsson added a comment -
        I have created something that seems to be working for me. It can definitely use more testing and further patches, suggestions and improvements are welcome. I have pushed the script to the consol-proto repo as well as included it in the patch set for consolidation. This script replaces the jdk 9 unshuffle script so if you have a patch for 10 that needs to go to 8, you will need to do it in two steps. First 10->9 using the script in 10, then 9->8 using the script in 9.

        http://hg.openjdk.java.net/jdk10/consol-proto/file/caa9d7f59d2c/bin/unshuffle_patch.sh
        Show
        erikj Erik Joelsson added a comment - I have created something that seems to be working for me. It can definitely use more testing and further patches, suggestions and improvements are welcome. I have pushed the script to the consol-proto repo as well as included it in the patch set for consolidation. This script replaces the jdk 9 unshuffle script so if you have a patch for 10 that needs to go to 8, you will need to do it in two steps. First 10->9 using the script in 10, then 9->8 using the script in 9. http://hg.openjdk.java.net/jdk10/consol-proto/file/caa9d7f59d2c/bin/unshuffle_patch.sh
        erikj Erik Joelsson made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Understanding Fix Understood [ 10001 ]
        Hide
        darcy Joe Darcy added a comment -
        The script in question was included as part of the push of JDK- 8187443.
        Show
        darcy Joe Darcy added a comment - The script in question was included as part of the push of JDK- 8187443.
        darcy Joe Darcy made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Understanding Fix Understood [ 10001 ]
        Resolution Other [ 6 ]

          People

          • Assignee:
            erikj Erik Joelsson
            Reporter:
            darcy Joe Darcy
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: