Skip to content

feat(mkapk): fix potential missing executable mode bits#59

Merged
Xe merged 2 commits intoTecharoHQ:mainfrom
kouhaidev:patch-5
Nov 23, 2025
Merged

feat(mkapk): fix potential missing executable mode bits#59
Xe merged 2 commits intoTecharoHQ:mainfrom
kouhaidev:patch-5

Conversation

@kouhaidev
Copy link
Copy Markdown
Contributor

if there was a strict mode, these should error. "do what i mean" is fine for now tho

Checklist

@kouhaidev kouhaidev force-pushed the patch-5 branch 2 times, most recently from 88fa769 to 942a889 Compare November 14, 2025 04:36
if there was a strict mode, these should error. "do what i mean" is fine for now tho

Signed-off-by: Kouhai <66407198+kouhaidev@users.noreply.github.com>
platform := fmt.Sprintf("%s/%s", goos, cpu)
t.Run(platform, func(t *testing.T) {
fname := yeettest.BuildHello(t, Build, yeettest.BuildHelloInput{
// yeettest manipulates the internal.PackageDestDir global, t.Parallel will race
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

#60

Comment thread internal/mkapk/mkapk_test.go Outdated
Signed-off-by: Xe Iaso <me@xeiaso.net>
@Xe Xe enabled auto-merge (squash) November 23, 2025 12:42
@Xe Xe merged commit fd6d23e into TecharoHQ:main Nov 23, 2025
9 checks passed
Xe pushed a commit that referenced this pull request Nov 23, 2025
# [0.8.0](v0.7.0...v0.8.0) (2025-11-23)

### Bug Fixes

* **mkapk:** rename fields to initd/confd for consistency with alpine-sdk ([#61](#61)) ([f194f53](f194f53)), closes [#58](#58)

### Features

* build confexts, portables, and sysexts ([#62](#62)) ([9aa3c2a](9aa3c2a))
* **mkapk:** expose apk packaging builtin ([#58](#58)) ([f08c123](f08c123))
* **mkapk:** fix potential missing executable mode bits ([#59](#59)) ([fd6d23e](fd6d23e))

Signed-Off-By: Mimi Yasomi <mimi@techaro.lol>
@Xe
Copy link
Copy Markdown
Contributor

Xe commented Nov 23, 2025

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants