Skip to content

maint(linux): use tc_rsync_upload and write_download_info functions 🪶#14258

Merged
ermshiperete merged 1 commit intomasterfrom
maint/linux/rsync
Jul 3, 2025
Merged

maint(linux): use tc_rsync_upload and write_download_info functions 🪶#14258
ermshiperete merged 1 commit intomasterfrom
maint/linux/rsync

Conversation

@ermshiperete
Copy link
Copy Markdown
Contributor

@ermshiperete ermshiperete commented Jul 2, 2025

Also make _tc_rsync function cross-platform.

A comparison might show that we omit --perms when calling rsync with this change. However the short form (-p) is already included, so we don't have to additional add the long form --perms.

Test-bot: skip

@ermshiperete ermshiperete requested a review from mcdurdin July 2, 2025 16:51
@github-project-automation github-project-automation bot moved this to Todo in Keyman Jul 2, 2025
@keymanapp-test-bot keymanapp-test-bot bot added the user-test-missing User tests have not yet been defined for the PR label Jul 2, 2025
@keymanapp-test-bot keymanapp-test-bot bot added this to the A19S6 milestone Jul 2, 2025
@github-actions github-actions bot added web/ windows/ android/ developer/ ios/ mac/ linux/ common/ core/ Keyman Core resources/ maint Maintenance work -- continuous integration, build scripts, infrastructure labels Jul 2, 2025
@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-missing User tests have not yet been defined for the PR label Jul 2, 2025
@ermshiperete ermshiperete changed the title maint(linux): use tc_rsync_upload and write_download_info functions maint(linux): use tc_rsync_upload and write_download_info functions 🪶 Jul 2, 2025
Also make `_tc_rsync` function cross-platform. A comparison might show
that we omit `--perms` when calling rsync with this change. However the
short form (`-p`) is already included, so we don't have to additional
add the long form `--perms`.
Copy link
Copy Markdown
Member

@mcdurdin mcdurdin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Base automatically changed from maint/android/14188_tcconfig to master July 3, 2025 15:48
@ermshiperete ermshiperete merged commit 9c9d48b into master Jul 3, 2025
29 checks passed
@ermshiperete ermshiperete deleted the maint/linux/rsync branch July 3, 2025 15:48
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Jul 3, 2025
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 19.0.76-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

android/ common/ core/ Keyman Core developer/ ios/ linux/ mac/ maint Maintenance work -- continuous integration, build scripts, infrastructure resources/ web/ windows/

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants