Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-16179

gen-doc depends on jar

Agile BoardAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskMoveLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Normal
    • Resolution: Duplicate
    • None
    • None
    • All
    • None

    Description

      The target 'gen-doc' depends on 'jar' so let's add it the depends attribute. Patches for trunk and 3.11 are attached.

      # ant gen-doc
      gen-doc:
           [exec] python convert_yaml_to_rst.py ../conf/cassandra.yaml source/configuration/cassandra_config_file.rst
           [exec] python gen-nodetool-docs.py
           [exec] ls: cannot access '../bin/../build/apache-cassandra*.jar': No such file or directory
           [exec] ERROR: Nodetool failed to run, you likely need to build cassandra using ant jar from the top level directory
           [exec] Error: Could not find or load main class org.apache.cassandra.tools.NodeTool
           [exec] Makefile:64: recipe for target 'html' failed
           [exec] Traceback (most recent call last):
           [exec]   File "gen-nodetool-docs.py", line 68, in <module>
           [exec]     create_help_file()
           [exec]   File "gen-nodetool-docs.py", line 52, in create_help_file
           [exec]     raise cpe
           [exec]   File "gen-nodetool-docs.py", line 46, in create_help_file
           [exec]     subprocess.check_call([nodetool, "help"], stdout=output_file)
           [exec]   File "/usr/lib/python3.6/subprocess.py", line 311, in check_call
           [exec]     raise CalledProcessError(retcode, cmd)
           [exec] subprocess.CalledProcessError: Command '['../bin/nodetool', 'help']' returned non-zero exit status 1.
           [exec] make: *** [html] Error 1
           [exec] Result: 2

      The then redundant 'jar' dependency on the 'artifacts' target could be removed too, though the attached patches don't include this modification.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            polo-language Angelo Polo Assign to me
            polo-language Angelo Polo
            Angelo Polo
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment