Accumulo
  1. Accumulo
  2. ACCUMULO-2646

Generate example configuration from template in build

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.1, 1.7.0
    • Component/s: build
    • Labels:
      None

      Description

      It was made clear in the mailing list discussion that there is value in providing static example configuration, for documentation purposes.

      As a follow-on to ACCUMULO-1395, it would be best if we generate this provided example configuration in the build, to save us the effort of maintaining separate version-controlled example files that are mostly redundant.

        Issue Links

          Activity

          Hide
          Bill Havanki added a comment -

          Attaching diff for change on top of patch 0001 for detecting Darwin (Mac OS X) and using a BSD-specific getopt in bootstrap_config.sh. Tested on Mavericks.

          Show
          Bill Havanki added a comment - Attaching diff for change on top of patch 0001 for detecting Darwin (Mac OS X) and using a BSD-specific getopt in bootstrap_config.sh. Tested on Mavericks.
          Hide
          Christopher Tubbs added a comment -

          It looks like the problem is that bootstrap_config.sh's command line arguments added in ACCUMULO-1395 uses GNU getopt, and that supports long opts, whereas BSD getopt does not. However, since the external script looks cleaner, I'm going to go ahead and apply that. A separate issue (ACCUMULO-2956) is opened to add non-GNU getopt support to bootstrap_config.sh

          Show
          Christopher Tubbs added a comment - It looks like the problem is that bootstrap_config.sh's command line arguments added in ACCUMULO-1395 uses GNU getopt, and that supports long opts, whereas BSD getopt does not. However, since the external script looks cleaner, I'm going to go ahead and apply that. A separate issue ( ACCUMULO-2956 ) is opened to add non-GNU getopt support to bootstrap_config.sh
          Hide
          Christopher Tubbs added a comment -
          Show
          Christopher Tubbs added a comment - Added 0001-ACCUMULO-2646-Move-example-config-generation-to-scri.patch for OS X users to review.
          Hide
          Christopher Tubbs added a comment -

          As reported by Bill Havanki, this change does not work in OS X.

          The for loop to generate the config should be moved to a script.

          Show
          Christopher Tubbs added a comment - As reported by Bill Havanki , this change does not work in OS X. The for loop to generate the config should be moved to a script.

            People

            • Assignee:
              Christopher Tubbs
              Reporter:
              Christopher Tubbs
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 1h
                1h

                  Development