Opened 9 years ago

Closed 8 years ago

#49575 closed defect (fixed)

py{27,34}-awscli @1.7.22_1 should depend on py{27,34}-docutils

Reported by: benzado (Benjamin Ragheb) Owned by: pixilla (Bradley Giesbrecht)
Priority: Normal Milestone:
Component: ports Version: 2.3.4
Keywords: Cc: larryv (Lawrence Velázquez), kurthindenburg (Kurt Hindenburg), rnowling@…
Port: py-awscli

Description

Installing py27-awscli @1.7.22_1 and running aws yields this output:

Traceback (most recent call last):
  File "/opt/local/bin/aws", line 19, in <module>
    import awscli.clidriver
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/awscli/clidriver.py", line 30, in <module>
    from awscli.help import ProviderHelpCommand
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/awscli/help.py", line 19, in <module>
    from docutils.core import publish_string
ImportError: No module named docutils.core

After port install py27-docutils, the output is:

usage: aws [options] <command> <subcommand> [parameters]
aws: error: too few arguments

So it appears that py27-awscli ought to depend on py27-docutils.

Change History (10)

comment:1 Changed 9 years ago by mf2k (Frank Schima)

Cc: pixilla@… removed
Owner: changed from macports-tickets@… to pixilla@…
Port: py-awscli added; py27-awscli removed

comment:2 Changed 9 years ago by joshenders (Josh Enders)

Experiencing this as well:

Traceback (most recent call last):
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/bin/aws", line 19, in <module>
    import awscli.clidriver
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/awscli/clidriver.py", line 30, in <module>
    from awscli.help import ProviderHelpCommand
  File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/awscli/help.py", line 19, in <module>
    from docutils.core import publish_string
ImportError: No module named docutils.core

Installing docutils fixes the issue:

sudo port install py27-docutils

Here's a patch:

Alphabetized the dependencies and adds py27-docutils

--- Portfile	2015-11-23 15:19:58.000000000 -0800
+++ Portfile.new	2015-11-23 15:17:24.000000000 -0800
@@ -30,13 +30,14 @@
     depends_build-append \
                         port:py${python.version}-setuptools
     depends_lib-append \
+                        port:py${python.version}-bcdoc \
                         port:py${python.version}-botocore \
-                        port:py${python.version}-six \
+                        port:py${python.version}-colorama \
                         port:py${python.version}-dateutil \
+                        port:py${python.version}-docutils \
                         port:py${python.version}-jmespath \
-                        port:py${python.version}-colorama \
-                        port:py${python.version}-bcdoc \
-                        port:py${python.version}-rsa
+                        port:py${python.version}-rsa \
+                        port:py${python.version}-six

     depends_run-append  port:awscli_select

comment:3 Changed 9 years ago by larryv (Lawrence Velázquez)

Cc: larryv@… added

Cc Me!

comment:4 Changed 9 years ago by larryv (Lawrence Velázquez)

Summary: py27-awscli @1.7.22_1 should depend on py27-docutilspy{27,34}-awscli @1.7.22_1 should depend on py{27,34}-docutils

This affects py34-awscli also.

comment:5 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:6 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

done r144867

comment:7 in reply to:  6 Changed 9 years ago by nareshov (Naresh V.)

Resolution: fixed
Status: closedreopened

Replying to khindenburg@…:

done r144867

This change doesn't include the necessary fix: adding docutils as a dependency.

comment:8 Changed 8 years ago by kurthindenburg (Kurt Hindenburg)

I don't recall why I didn't add docutils - but I don't get the issue now - do you?

comment:9 Changed 8 years ago by larryv (Lawrence Velázquez)

Cc: rnowling@… added

Has duplicate #52093. This is still unresolved; the py-docutils dependency is required.

comment:10 Changed 8 years ago by larryv (Lawrence Velázquez)

Resolution: fixed
Status: reopenedclosed

Fixed for real in r153585.

Note: See TracTickets for help on using tickets.