Uploaded image for project: 'Lucene - Core'
  1. Lucene - Core
  2. LUCENE-10173

remove "--max-workers 1" restriction when using external GPG w/gradle build

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 9.0
    • None
    • None
    • New

    Description

      LUCENE-9488 amended the existing support for using gradle's signing plugin with external GPG process to sign files, by adding support for the signing plugin's java-implementation, and switching to the pure-java implementation by default.

      In the process dawid missunderstood some of documented issues that can pop up when using an external GPG process, and thought that using an external GPG process could only work when using --max-workers 1.

      As a result, when changing the build so that using an external GPG process was optional via 'useGpg' he added an explicit check to fail the build if this option is used w/o --max-workers 1.

      This issue is to:

      • revert that restriction when 'useGpg' is specified since it isn't necessary (nor as it turns out, does it actually help)
      • add a new restriction when 'useGpg' is specified to fail fast if the required keyName isn't provided (this was handled in a differnet way prior to LUCENE-9488, but that logic was removed when 'useGpg' became optional – we should add it back if and only if 'useGpg' is specified to prevent a very confusing and missleading error message that can occur otherwise)
      • improve the associated docs

      Attachments

        1. LUCENE-10173.patch
          8 kB
          Chris M. Hostetter

        Issue Links

          Activity

            People

              hossman Chris M. Hostetter
              hossman Chris M. Hostetter
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Slack

                  Issue deployment