Skip to content

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen force-pushed the bugfix/809/map-de_DE-to-de-in-xcstrings branch from 4fcdf2e to ada387f Compare August 1, 2025 12:11
Copy link
Contributor

@i2h3 i2h3 left a comment

Choose a reason for hiding this comment

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

When I run this command on macOS, then the source file is truncated to zero byte content.

cat Sources/SwiftNextcloudUI/Localizable.xcstrings | jq '. | del(.strings .[] .localizations .de)' | sed 's/"de_DE": {/"de": {/' > Sources/SwiftNextcloudUI/Localizable.xcstrings

Removing the last part and letting it print to stdout works. I assume this is because of minor differences between GNU and BSD command line utilities.

@nickvergessen
Copy link
Member Author

When I run this command on macOS, then the source file is truncated to zero byte content.

Also happened online, so reverting the image for now:
nextcloud/swiftnextcloudui@43d2ba3

Seems you can not pipe into a file you are reading. So we need to pipe to a temp file and afterwards rename

@nickvergessen
Copy link
Member Author

nextcloud/swiftnextcloudui@dd776b3

Sync worked

@nickvergessen nickvergessen merged commit 5b07fb7 into master Aug 1, 2025
4 checks passed
@nickvergessen nickvergessen deleted the bugfix/809/map-de_DE-to-de-in-xcstrings branch August 1, 2025 14:00
@nickvergessen
Copy link
Member Author

Reverting for now, as due to nextcloud/swiftnextcloudui#3 repeated runs clear the german translations on transifex

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants