Skip to content

Commit

Permalink
fix publishing for docker auth & pypi name
Browse files Browse the repository at this point in the history
  • Loading branch information
AndreyNikiforov committed Jul 6, 2023
1 parent 475f9c4 commit 36b8efa
Show file tree
Hide file tree
Showing 9 changed files with 24 additions and 14 deletions.
8 changes: 7 additions & 1 deletion .github/workflows/create-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -184,6 +184,12 @@ jobs:
- name: Available platforms
run: echo ${{ steps.buildx.outputs.platforms }}

- name: Login to DockerHub
uses: docker/login-action@v2
with:
username: ${{ secrets.DOCKERHUB_USERNAME }}
password: ${{ secrets.DOCKERHUB_PASSWORD }}

- name: Build Release Docker and Publish
uses: docker/build-push-action@v3
with:
Expand Down Expand Up @@ -269,7 +275,7 @@ jobs:
python3 -m twine upload --non-interactive --disable-progress-bar dist/*.whl
release:
needs: [get_version, build_src,build_linux,build_macos,build_windows]
needs: [get_version,build_src,build_linux,build_macos,build_windows]
runs-on: ubuntu-20.04
steps:
- name: Checkout code
Expand Down
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

## 1.14.4 (2023-07-06)

- fix: docker auth during publishing

## 1.14.3 (2023-07-06)

- add binary wheel without dependencies to pypi
Expand Down
6 changes: 3 additions & 3 deletions EXPERIMENTAL.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,17 +34,17 @@ Legacy command (compatible with prior versions):

`docker run -it --rm icloudpd:icloudpd icloud --help`

`icloudpd-1.14.3-windows-amd64 --help`
`icloudpd-1.14.4-windows-amd64 --help`

Help:

`docker run -it --rm icloudpd:icloudpd`

`icloudpd-ex-1.14.3-windows-amd64 --help`
`icloudpd-ex-1.14.4-windows-amd64 --help`

Example:

`docker run -it --rm icloudpd:icloudpd copy my@email.address /path/to/{album}/{date_created:%Y/%Y-%m}`

`icloudpd-ex-1.14.3-windows-amd64 copy my@email.address /path/to/{album}/{date_created:%Y/%Y-%m}`
`icloudpd-ex-1.14.4-windows-amd64 copy my@email.address /path/to/{album}/{date_created:%Y/%Y-%m}`

10 changes: 5 additions & 5 deletions FAQ.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,10 +59,10 @@ iOS 16 feature to share libraries between accounts is [not supported](https://gi

Here are the steps to make it working:
- download binary from Github [Releases](https://github.com/icloud-photos-downloader/icloud_photos_downloader/releases) into desired local folder
- add executable flag by running `chmod +x icloudpd-1.14.3-macos-amd64`
- start it from the terminal: `icloudpd-1.14.3-macos-amd64`
- add executable flag by running `chmod +x icloudpd-1.14.4-macos-amd64`
- start it from the terminal: `icloudpd-1.14.4-macos-amd64`
- Apple will tell you that it cannot check for malicous software and refuse to run the app; click "Ok"
- Open "System Settings"/"Privacy & Security" and find `icloudpd-1.14.3-macos-amd64` as blocked app; Click "Allow"
- Start `icloudpd-1.14.3-macos-amd64` from the terminal again
- Open "System Settings"/"Privacy & Security" and find `icloudpd-1.14.4-macos-amd64` as blocked app; Click "Allow"
- Start `icloudpd-1.14.4-macos-amd64` from the terminal again
- Apple will show another warning; click "Open"
- After that you can run `icloudpd-1.14.3-macos-amd64 icloudpd --help` or any other supported command/option
- After that you can run `icloudpd-1.14.4-macos-amd64 icloudpd --help` or any other supported command/option
2 changes: 1 addition & 1 deletion binary_dist/pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ requires = [
build-backend = "setuptools.build_meta"

[project]
version="1.14.3"
version="1.14.4"
name = "icloudpd"
description = "icloudpd is a command-line tool to download photos and videos from iCloud."
readme = "README_PYPI.md"
Expand Down
2 changes: 1 addition & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ requires = [
build-backend = "setuptools.build_meta"

[project]
version="1.14.3"
version="1.14.4"
name = "icloudpd"
description = "icloudpd is a command-line tool to download photos and videos from iCloud."
readme = "README_PYPI.md"
Expand Down
2 changes: 1 addition & 1 deletion scripts/build_binary_dist_macos
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ cp dist/icloudpd-$1-macos-amd64 dist/binary_dist/src/icloudpd/icloudpd
cp dist/icloud-$1-macos-amd64 dist/binary_dist/src/icloud/icloud
python3 -m pip wheel dist/binary_dist/ -w dist/binary_dist --no-deps --no-build-isolation

mv -f dist/binary_dist/icloudpd-$1-py2.py3-none-any.whl dist/icloudpd-$1-py2.py3-none-macos_11_x86_64.macos_arm64.whl
mv -f dist/binary_dist/icloudpd-$1-py2.py3-none-any.whl dist/icloudpd-$1-py2.py3-none-macos_11_0_x86_64.macos_11_0_arm64.whl
2 changes: 1 addition & 1 deletion scripts/publish_pypi
Original file line number Diff line number Diff line change
Expand Up @@ -11,4 +11,4 @@ set -euo pipefail
# cleint use: pip install --index http://192.168.1.20:4880 --trusted-host 192.168.1.20 icloudpd

# linux
twine upload --repository-url $1 --non-interactive -u bogus -p bogus dist/icloudpd-1.14.3-py2.py3-none-manylinux*.whl dist/icloudpd-1.14.3-py2.py3-none-macos*.whl dist/icloudpd-1.14.3-py2.py3-none-win*.whl dist/icloudpd-1.14.3-py3-none-any.whl
twine upload --repository-url $1 --non-interactive -u bogus -p bogus dist/icloudpd-1.14.4-py2.py3-none-manylinux*.whl dist/icloudpd-1.14.4-py2.py3-none-macos*.whl dist/icloudpd-1.14.4-py2.py3-none-win*.whl dist/icloudpd-1.14.4-py3-none-any.whl
2 changes: 1 addition & 1 deletion src/icloudpd/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@
type=click.IntRange(1),
)
# a hacky way to get proper version because automatic detection does not work for some reason
@click.version_option(version="1.14.3")
@click.version_option(version="1.14.4")
# pylint: disable-msg=too-many-arguments,too-many-statements
# pylint: disable-msg=too-many-branches,too-many-locals
def main(
Expand Down

0 comments on commit 36b8efa

Please sign in to comment.