Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Development #139

Merged
merged 29 commits into from
Jul 21, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
51444d8
Bump poetry from 1.4.2 to 1.5.1 in /.github/workflows
dependabot[bot] May 30, 2023
a08ce8f
Bump pre-commit from 3.2.2 to 3.3.3
dependabot[bot] Jun 14, 2023
03c4c04
Bump ruamel-yaml from 0.17.21 to 0.17.32
dependabot[bot] Jun 19, 2023
5663772
Bump mypy from 1.1.1 to 1.4.1
dependabot[bot] Jun 26, 2023
f55952d
Bump release-drafter/release-drafter from 5.23.0 to 5.24.0
dependabot[bot] Jun 28, 2023
43cc8e0
Bump pyupgrade from 3.3.1 to 3.9.0
dependabot[bot] Jul 10, 2023
0467d3f
Bump flake8-bugbear from 23.3.23 to 23.7.10
dependabot[bot] Jul 11, 2023
0094013
Bump pypa/gh-action-pypi-publish from 1.8.5 to 1.8.8
dependabot[bot] Jul 12, 2023
90142fb
Bump pip from 23.0.1 to 23.2 in /.github/workflows
dependabot[bot] Jul 17, 2023
7ad3e0b
Bump virtualenv from 20.21.0 to 20.24.1 in /.github/workflows
dependabot[bot] Jul 20, 2023
4e0ca5e
Merge pull request #119 from jyksnw/dependabot/pip/dot-github/workflo…
jyksnw Jul 20, 2023
0dcb97d
Merge pull request #123 from jyksnw/dependabot/pip/pre-commit-3.3.3
jyksnw Jul 20, 2023
31e0d19
Merge branch 'development' into dependabot/pip/ruamel-yaml-0.17.32
jyksnw Jul 20, 2023
d35f216
Merge pull request #124 from jyksnw/dependabot/pip/ruamel-yaml-0.17.32
jyksnw Jul 20, 2023
0b5bfab
Merge branch 'development' into dependabot/pip/mypy-1.4.1
jyksnw Jul 20, 2023
a5a5c07
Merge pull request #128 from jyksnw/dependabot/pip/mypy-1.4.1
jyksnw Jul 20, 2023
d94a525
Merge pull request #130 from jyksnw/dependabot/github_actions/release…
jyksnw Jul 20, 2023
605489d
Merge branch 'development' into dependabot/pip/pyupgrade-3.9.0
jyksnw Jul 20, 2023
b021457
Merge pull request #132 from jyksnw/dependabot/pip/pyupgrade-3.9.0
jyksnw Jul 20, 2023
ef84e0d
Merge branch 'development' into dependabot/pip/flake8-bugbear-23.7.10
jyksnw Jul 21, 2023
0bc6548
Merge pull request #133 from jyksnw/dependabot/pip/flake8-bugbear-23.…
jyksnw Jul 21, 2023
1f549ba
Merge pull request #134 from jyksnw/dependabot/github_actions/pypa/gh…
jyksnw Jul 21, 2023
cb83b54
Merge branch 'development' into dependabot/pip/dot-github/workflows/p…
jyksnw Jul 21, 2023
2641a7c
Merge pull request #137 from jyksnw/dependabot/pip/dot-github/workflo…
jyksnw Jul 21, 2023
8d27107
Merge branch 'development' into dependabot/pip/dot-github/workflows/v…
jyksnw Jul 21, 2023
9b129c5
Merge pull request #138 from jyksnw/dependabot/pip/dot-github/workflo…
jyksnw Jul 21, 2023
527ba6a
Fixes #135 - Adds an edgecase to handle MacOS on ARM
jyksnw Jul 21, 2023
0bdc610
Fixes #97 - Removes the use of deprecated modules and methods
jyksnw Jul 21, 2023
d3e94b4
bump version for relase
jyksnw Jul 21, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .github/workflows/constraints.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
pip==23.0.1
poetry==1.4.2
virtualenv==20.21.0
pip==23.2
poetry==1.5.1
virtualenv==20.24.1
6 changes: 3 additions & 3 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -57,21 +57,21 @@ jobs:

- name: Publish package on PyPI
if: steps.check-version.outputs.tag
uses: pypa/gh-action-pypi-publish@v1.8.5
uses: pypa/gh-action-pypi-publish@v1.8.8
with:
user: __token__
password: "${{ secrets.PYPI_TOKEN }}"

- name: Publish package on TestPyPI
if: "! steps.check-version.outputs.tag"
uses: pypa/gh-action-pypi-publish@v1.8.5
uses: pypa/gh-action-pypi-publish@v1.8.8
with:
user: __token__
password: "${{ secrets.TEST_PYPI_TOKEN }}"
repository_url: https://test.pypi.org/legacy/

- name: Publish the release notes
uses: release-drafter/release-drafter@v5.23.0
uses: release-drafter/release-drafter@v5.24.0
with:
publish: "${{ steps.check-version.outputs.tag != '' }}"
tag: "${{ steps.check-version.outputs.tag }}"
Expand Down
5 changes: 5 additions & 0 deletions jdk/client/adoptium.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,11 @@ def get_download_url(

if arch is None:
arch = Architecture.detect()
arch = AdoptiumArchitecture.transform(arch)

# Handle edge case for MacOS w/Apple M1 or M2
if operating_system is OperatingSystem.MAC and arch == AdoptiumArchitecture.ARM:
arch = AdoptiumArchitecture.AARCH64

if jre:
image_type = ImageType.JRE
Expand Down
11 changes: 4 additions & 7 deletions jdk/client/client.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import cgi
import shutil
import tempfile
from collections.abc import Iterable
Expand Down Expand Up @@ -52,12 +51,10 @@ def download(self, download_url: str) -> Optional[str]:

jdk_file = None
with request.urlopen(req) as open_request: # noqa: S310
info = open_request.info()
if "Content-Disposition" in info:
content_disposition = info["Content-Disposition"]
_, params = cgi.parse_header(content_disposition)
if "filename" in params:
jdk_file = params["filename"]
headers = open_request.headers
content_disposition = headers.get_content_disposition()
if content_disposition:
jdk_file = headers.get_filename()
else:
url_path = urlsplit(download_url).path
jdk_file = path.basename(url_path)
Expand Down
Loading