Skip to content

build: package for nix flake - #39

Draft
Vuks69 wants to merge 1 commit into
mainfrom
flake-package
Draft

Vuks69 wants to merge 1 commit into
mainfrom
flake-package

Conversation

@Vuks69

@Vuks69 Vuks69 commented Sep 11, 2026

Copy link
Copy Markdown
Member

Adds default flake package.
Allows developers to build using nix build
Built with further packages under cmd/ in mind - simply add the new package in packages = ...;
Central version definition - on build it is injected into the variables in core. For further use with eg. debug or help commands.
Vendor hash is derived from go.mod/go.sum, so unless a cmd/... package needs a special import, it'll be the same for all packages derived from this repo.

@analeis analeis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, lets merge

@Vuks69

Vuks69 commented Sep 27, 2026

Copy link
Copy Markdown
Member Author

as it is right now the packaging is fairly rudimentary, i'm working on a better design that includes proper nix options and linking postgres as a dependency

@Vuks69
Vuks69 marked this pull request as draft September 27, 2026 14:14
@inomish

inomish commented Oct 24, 2026

Copy link
Copy Markdown
Contributor

@Vuks69 I made a PR that moves from flakelight, and just generally cleans up some old stuff, can you maybe move the package it there then I'll write a review

@Vuks69
Vuks69 force-pushed the flake-package branch 2 times, most recently from 427671a to f9b7135 Compare October 28, 2026 16:22
@Vuks69

Vuks69 commented Oct 28, 2026

Copy link
Copy Markdown
Member Author

rebased to include the flakelight drop, waiting for future postgres drop as well

@Vuks69
Vuks69 force-pushed the flake-package branch 3 times, most recently from 4fbf6f9 to 7122dac Compare November 7, 2026 17:35
@Vuks69

Vuks69 commented Nov 7, 2026

Copy link
Copy Markdown
Member Author

Currently unsure about the container image building/argument passing, to ensure the version/commit info is included there.

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.

3 participants