Skip to content

Commit

Permalink
fix(python): incorrect dependency on jsii (#1582)
Browse files Browse the repository at this point in the history
The Python runtime dependency of generated Python code was incorrectly pinning on the minor, instead of allowing the necessary semver range. Changed the dependency statement from `~=<version>` to the correctly translated caret range.

Fixes #1573
  • Loading branch information
RomainMuller committed Apr 20, 2020
1 parent fac53d9 commit eed53ed
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion packages/jsii-pacmak/lib/targets/python.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1212,7 +1212,7 @@ class Package {
packages: modules.map(m => m.pythonName),
package_data: packageData,
python_requires: '>=3.6',
install_requires: [`jsii~=${jsiiVersionSimple}`, 'publication>=0.0.3'].concat(dependencies),
install_requires: [`jsii${toPythonVersionRange(`^${jsiiVersionSimple}`)}`, 'publication>=0.0.3'].concat(dependencies),
classifiers: [
'Intended Audience :: Developers',
'Operating System :: OS Independent',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
},
"python_requires": ">=3.6",
"install_requires": [
"jsii~=1.1.0",
"jsii>=1.1.0, <2.0.0",
"publication>=0.0.3"
],
"classifiers": [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
},
"python_requires": ">=3.6",
"install_requires": [
"jsii~=1.1.0",
"jsii>=1.1.0, <2.0.0",
"publication>=0.0.3",
"scope.jsii-calc-base-of-base>=1.1.0, <2.0.0"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
},
"python_requires": ">=3.6",
"install_requires": [
"jsii~=1.1.0",
"jsii>=1.1.0, <2.0.0",
"publication>=0.0.3",
"scope.jsii-calc-base>=1.1.0, <2.0.0"
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
},
"python_requires": ">=3.6",
"install_requires": [
"jsii~=1.1.0",
"jsii>=1.1.0, <2.0.0",
"publication>=0.0.3",
"scope.jsii-calc-base>=1.1.0, <2.0.0",
"scope.jsii-calc-base-of-base>=1.1.0, <2.0.0",
Expand Down

0 comments on commit eed53ed

Please sign in to comment.