Skip to content

+aerc#12471

Merged
jhheider merged 3 commits intopkgxdev:mainfrom
scottjg:aerc
Apr 5, 2026
Merged

+aerc#12471
jhheider merged 3 commits intopkgxdev:mainfrom
scottjg:aerc

Conversation

@scottjg
Copy link
Copy Markdown
Contributor

@scottjg scottjg commented Apr 4, 2026

Summary

  • What changed:

    Added new package aerc-mail.org to the pantry. aerc is a terminal email client supporting IMAP, SMTP, Maildir, JMAP, and more.

  • Why:

    aerc was not available in the pkgx pantry.

AI Intent

  • Prompt or task statement:

    Add the aerc package to the pkgx pantry, referencing the Homebrew formula for build details.

  • Scope of AI-assisted changes:

    AI authored the projects/aerc-mail.org/package.yml file with human review.

Risk Assessment

  • Risk level:

    low

  • Primary risks:

    None significant. this is a new standalone package with no downstream dependents.

  • Compatibility impact:

    None — additive only.

Rollback Plan

  • Revert path:

    Revert the single commit removing projects/aerc-mail.org/.

  • Forward-fix path (if revert is not possible):

    N/A

Validation

  • Local checks run:

    bk build aerc-mail.org succeeded on macOS arm64, producing a working aerc v0.21.0 binary.

  • CI checks expected:

    Standard pantry CI build + test on Linux and macOS.

Internal/Public Boundary Check

  • No internal-only data, private URLs, secrets, or private runbooks were
    added.

@scottjg scottjg changed the title feat(aerc-mail.org): add aerc terminal email client +aerc Apr 4, 2026
aerc bakes its share and libexec directories into the binary at compile
time via Go ldflags. Without explicit overrides, these resolve to the
build machine's absolute prefix (including the +brewing suffix), which
won't exist on other users' machines or even locally after pkgx moves
the build to its final path.

Fix by setting SHAREDIR and LIBEXECDIR to ~/.pkgx/aerc-mail.org/v<ver>/
paths with a literal tilde. aerc's xdg.ExpandHome() expands ~ to the
current user's home directory at runtime, so the binary correctly
locates its config files on any machine.
@scottjg
Copy link
Copy Markdown
Contributor Author

scottjg commented Apr 5, 2026

so this latest commit "works" but i'm not sure about the best pattern for fixing issues like this where the prefix is baked into the binary.

Updated Go version dependency and modified build script to use MAKE_ARGS for installation.
@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented Apr 5, 2026

thanks, @scottjg . cleaned up the syntax a little, but it looks good.

@jhheider jhheider merged commit e426d6a into pkgxdev:main Apr 5, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants