about summary refs log tree commit diff
AgeCommit message (Collapse)AuthorFilesLines
2021-09-21 r/2898 chore(3p/cgit): git: update to v2.30.1Christian Hesse1-1/+1
Update to git version v2.30.1, no additional changes required. Signed-off-by: Christian Hesse <mail@eworm.de> Change-Id: I9c8a572fcef2dbb3164ecc397dc071240eba8480
2021-09-21 r/2897 test(3p/cgit): t0107: support older and/or non-GNU tarTodd Zullinger1-4/+4
The untar tests for various compression algorithms use shortcut options from GNU tar to handle decompression. These options may not be provided by non-GNU tar nor even by slightly older GNU tar versions which ship on many systems. An example of the latter case is the --zstd option. This was added in GNU tar-1.32 (2019-02-23)¹. This version of tar is not provided by CentOS/RHEL, in particular. In Debian, --zstd has been backported to the tar-1.30 release. Avoid the requirement on any specific implementations or versions of tar by piping decompressed output to tar. This is compatible with older GNU tar releases as well as tar implementations from other vendors. (It may also be a slight benefit that this more closely matches what the snapshot creation code does.) ¹ Technically, the --zstd option was first released in tar-1.31 (2019-01-02), but this release was very short-lived and is no longer listed on the GNU Tar release page. Signed-off-by: Todd Zullinger <tmz@pobox.com> Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Change-Id: Ib65412cd1d23312ddd4cf840c09efc32512d3122
2021-09-21 r/2896 refactor(3p/cgit): md2html: use sane_lists extensionJason A. Donenfeld1-0/+1
This allows for cleaner nesting semantics and matches github more closely. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> Change-Id: I93dacc0c276fefb28d67379ef38b8647c584244b
2021-09-21 r/2895 chore(3p/cgit): git: update to v2.30.0Christian Hesse5-33/+35
Update to git version v2.30.0, this requires changes for these upstream commits: * 88894aaeeae92e8cb41143cc2e045f50289dc790 blame: simplify 'setup_scoreboard' interface * 1fbfdf556f2abc708183caca53ae4e2881b46ae2 banned.h: mark non-reentrant gmtime, etc as banned Signed-off-by: Christian Hesse <mail@eworm.de> Change-Id: I6076ef250102010b601c92e9ea5bab2061b77006
2021-09-21 r/2894 chore(3p/cgit): git: update to v2.29.2Christian Hesse1-1/+1
Update to git version v2.29.2. No changes required. Signed-off-by: Christian Hesse <mail@eworm.de> Change-Id: I8a3c12fdaa492ede79a750eea4b78f750dbfa18f
2021-09-21 r/2893 chore(3p/cgit): git: update to v2.29.1Christian Hesse2-2/+1
Update to git version v2.29.1. No functional change, but we want latest and greated version number, no? 😜 Signed-off-by: Christian Hesse <mail@eworm.de> Change-Id: Ie890ace2e88dca0d7defb1cac5a2723699f794db
2021-09-20 r/2892 fix(3p/overlays/tvl): Add fix for Steam web view issueVincent Ambo1-0/+5
See https://github.com/NixOS/nixpkgs/issues/137279 Change-Id: Id47f87396ad5e46c98080c7910d337d2643bcbd5 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3604 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-20 r/2891 refactor(depot): Import depot.path using lib.cleanSourceVincent Ambo1-1/+1
This imports a copy of the depot as `depot.path` without including the .git directory and other stuff that isn't tracked in the tree. Significantly reduces thie amount of data copied into the store. Change-Id: I567c0f969d1cea81d121588548f6db627c8f3432 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3601 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-18 r/2890 refactor(whitby): Move restic path configuration into modulesVincent Ambo4-7/+8
This lets each service declare their backup paths together with the configuration for the service, which is a lot more sensible than what we had before. Fixes b/147 Change-Id: If76fe62639f4cc0e6fbb63a2959d584479d8f0fb Reviewed-on: https://cl.tvl.fyi/c/depot/+/3583 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-18 r/2889 fix(ops/restic): types.string -> types.strVincent Ambo1-3/+3
I can never remember which is which. Change-Id: I69b8235862b8c5b49030a74bfca25aaa113273b7 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3582 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-18 r/2888 feat(besadii): Link to started builds in CL commentsVincent Ambo1-5/+35
This makes it easier to click through to a build from Gerrit after submitting a CL. Change-Id: Ic5c6eeb81c87bc4ea23c5c5ca25704434b081fd0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3572 Tested-by: BuildkiteCI Reviewed-by: lukegb <lukegb@tvl.fyi>
2021-09-18 r/2887 refactor(besadii): Extract logic for posting review commentsVincent Ambo1-42/+50
Currently besadii only posts comments when builds succeed, but it might be very useful to also have a link to a build when the build is started. This just shuffles code around. The only functional change is that the `labels` field in the review input is marked as `omitempty`, as this will not be needed when posting the build start comment. Change-Id: Id4a43fad8817c9a15da02f01ab2b781d48b46978 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3571 Tested-by: BuildkiteCI Reviewed-by: lukegb <lukegb@tvl.fyi>
2021-09-18 r/2886 fix(web/tvl): consistently uppercase Profpatschsterni1-2/+2
Him and me floating on our own is kinda cute, but breaks the graph. Change-Id: I043b327172781447bd0ce923e78e72f34bbff41b Reviewed-on: https://cl.tvl.fyi/c/depot/+/3573 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-09-18 r/2885 docs(kontemplate): Remove mention of kontemplate websiteVincent Ambo1-3/+2
This doesn't exist anymore. Change-Id: I4535e056acba3bbc7bbd1e764a0b3043639b0877 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3570 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-09-18 r/2884 feat(tazjin/gruber-darker): Check in first version of Quassel themeVincent Ambo1-0/+508
Change-Id: I5a79f4d6ccd8ddae4a6e24c356d4b7498ffb3c49 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3569 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-18 r/2883 style(tazjin/emacs): Reenable my original dark themeVincent Ambo2-5/+24
There's no longer an Egyptian fireball in the sky, so I can go back to normal. Change-Id: I6fdcd12f3d3e62c367115f3712cc0fd36eeff78d Reviewed-on: https://cl.tvl.fyi/c/depot/+/3568 Reviewed-by: tazjin <mail@tazj.in> Tested-by: BuildkiteCI
2021-09-18 r/2882 feat(nix/nint): accept attribute set with stdout, stderr and exitsterni2-3/+57
This extends the calling convention for nint in a non-breaking way: If the called script returns an attribute set instead of a string the following is done: * If the attributes `stdout` and/or `stderr` exist, their content (which must be a string currently) is written to the respective output. * If the attribute `exit` exists, nint will exit with the given exit code. Must be a number that can be converted to an `i32`. If it's missing, nint will exit without indicating an error. Change-Id: I209cf178fee3d970fdea3b26e4049e944af47457 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3547 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-17 r/2881 refactor(nix/runTestsuite): clean up runTestsuitesterni1-16/+6
* goodAss wasn't used before. Simplify it to just return a boolean, so we can use it for partitionTests later. * goodIt also returns unnecessary extra meta information which is not used. Cleaning that up makes the condition extremely small, so we can inline it into (what was) goodIts. * goodIts is just called in one place, so we can inline it into res. Change-Id: I70cf4fa3f61ce1467a2ee5319f841cdd42db6a66 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3548 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-09-17 r/2880 test(nix/utils): unit test storePathNamesterni1-0/+16
Change-Id: I4208cf9e3c5e1d922ee5b5bffd034e4ac6d0e2c0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3543 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-16 r/2879 docs(kontemplate): Update cloning docs in READMEVincent Ambo1-4/+4
Change-Id: I42bf2524650bf09104e48c1c1a54c97f3470b628 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3566 Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: flokli <flokli@flokli.de> Tested-by: BuildkiteCI
2021-09-16 r/2878 refactor(ops/restic): Move restic configuration into a new moduleVincent Ambo2-22/+75
Relates to b/147. First step towards giving depot modules the ability to declare their own backup directories by moving all restic configuration into a new module and adding a NixOS option for inclusion/exclusion paths for backups. This still keeps all backup paths within the whitby config. Change-Id: Ia96833668f1a3d02da892261153d8b02156b8ac0 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3565 Tested-by: BuildkiteCI Reviewed-by: flokli <flokli@flokli.de>
2021-09-16 r/2877 feat(git-serving): Configure josh to serve the depot over HTTPVincent Ambo4-17/+73
Previously we served the dumb git HTTP protocol from code.tvl.fyi via cgit. This CL disables this feature and instead runs josh in the same location (by redirecting appropriately), but while also enabling partial cloning of all subtrees of the depot. For example, after this CL the following would result in an independent clone of //nix/readTree: git clone https://code.tvl.fyi/depot.git:/nix/readTree.git Note that there are no josh workspaces configured at all for now, these references are only for static depot subpaths. Please refer to the documentation for josh for more information on available kinds of josh filters. Josh state is kept in a systemd state directory in /var/lib/josh and backed up to Restic. Backing this up is necessary, as josh uses stateful information to do things like tracking merges and rewriting history per subtree appropriately to avoid cloned repositories ending up in peculiar states. Change-Id: I156f0298c2aa42e3bdbf5a0e86109070d640c56e Reviewed-on: https://cl.tvl.fyi/c/depot/+/3563 Tested-by: BuildkiteCI Reviewed-by: flokli <flokli@flokli.de>
2021-09-16 r/2876 chore(cgit-taz): Disable HTTP git servingVincent Ambo1-2/+2
We don't need this anymore as josh will be doing it instead. Change-Id: I04324324fefa2b44604a8a5fad4dd3c7b7fe97a3 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3564 Tested-by: BuildkiteCI Reviewed-by: flokli <flokli@flokli.de>
2021-09-16 r/2875 refactor(3p/exwm): Remove `exwm-input--skip-buffer-list-update'Adrián Medraño Calvo2-9/+2
* exwm-input.el: (exwm-input--on-buffer-list-update): Stop checking `exwm-input--skip-buffer-list-update'; it's no longer needed now that we keep track of the last selected window and buffer. (exwm-input--skip-buffer-list-update): Remove variable. * exwm-manage.el (exwm-manage--manage-window): Remove binding of `exwm-input--skip-buffer-list-update'. Imported from https://github.com/ch11ng/exwm/pull/737 Change-Id: Iacbdb8b607a82352e5b55509b0aba93f1d87a9f2 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3562 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-16 r/2874 refactor(3p/exwm): Improve detection of focus changesQiang Fang1-7/+27
* exwm-input.el: (exwm-input--on-buffer-list-update): Keep track of last selected window and buffer, update focus only when any of those changes. (exwm-input--update-focus-defer): Add commentary. (exwm-input--update-focus-window-buffer): Add variable. Imported from https://github.com/ch11ng/exwm/pull/737 Copyright-paperwork-exempt: yes Co-Author: Adrián Medraño Calvo <adrian@medranocalvo.com> Change-Id: I3e53bcf45f04d0f9a88b757dffefe6de20daadfb Reviewed-on: https://cl.tvl.fyi/c/depot/+/3561 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-16 r/2873 chore(3p/josh): Bump version to latest masterVincent Ambo1-2/+2
Change-Id: I01b58653a9d0067deb9644ecfa4d4b3bd75185a9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3557 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-16 r/2872 fix(3p/exwm): Use hash table to cache ‘exwm-workspace--client-p’Matt Beshara1-2/+11
Commit imported from https://github.com/ch11ng/exwm/pull/848 Change-Id: I2b5509649b6933b52ed5df7f7fad7f3720f5facd Reviewed-on: https://cl.tvl.fyi/c/depot/+/3560 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-16 r/2871 chore(3p/overlays): Override EXWM sources with tree from depotVincent Ambo1-0/+5
Change-Id: I6047ab634deb2f019bcdc9a76a2194997fc84b29 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3559 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-16 r/2870 Merge "chore(3p/exwm): Subtree EXWM ... again" into canontazjin16-0/+8567
2021-09-15 r/2869 feat(nix/readTree): record list of children added by readTreesterni2-19/+35
This change adds a new attribute to readTree nodes, `__readTreeChildren` which is a list of attribute names added to this node by readTree. This is then used by `gather` for `ci.targets` to avoid evaluating attributes unnecessarily. Especially since Nix is not as lazy as we'd like when determining types (i. e. child ? __readTree needs to force `child` even when it's not an attribute set), evaluating attributes unnecessarily is sometimes problematic. Change-Id: I0a98691d41f987e23ee7e9ba21fbe465da5fe402
2021-09-15 r/2868 refactor(rust-crates-advisory): check type instead of blacklistingsterni1-3/+2
`our-crates` can just check if the attributes in question are derivation (i. e. have an `outPath`) instead of blacklisting the `__readTree` attribute specifically. Change-Id: I472692e89c0e9eff551372c72a73ab765b0b6599
2021-09-15 chore(3p/exwm): Subtree EXWM ... againVincent Ambo16-0/+8567
Upstream EXWM is now sort-of unmaintained[0], but some important PRs are pending[1] which I would like to absorb here somehow. [0]: https://github.com/ch11ng/exwm/issues/845 [1]: https://github.com/ch11ng/exwm/pull/848 git-subtree-dir: third_party/exwm git-subtree-mainline: 7894d7e178ac542f2542b71b10cb34e99080acd7 git-subtree-split: 0368127976bda29d35eed788edfe74644ecd3845 Change-Id: I078358c0ea1d2bac99111748ce9c137d2804ac4e
2021-09-15 r/2867 feat(3p/cgit): Highlight cheddar callout in about viewsVincent Ambo1-0/+26
Change-Id: Ica055ab621608ff78dbbf9ea9a2c9d1cdf2ff21e Reviewed-on: https://cl.tvl.fyi/c/depot/+/3556 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-15 r/2866 refactor(web/tvl): translate to markdownsterni1-15/+21
This is mostly equivalent, but we need to accomodate cheddar a bit: * cheddar doesn't like markdown inside of HTML, so the <main> tag around the image needs to go. * cheddar messes with a top-level SVG for some reason, so we need to wrap it in a <div> in order to prevent that. Change-Id: If9ed516623e81e24f600ee9f1b6d4d611b5bcedd Reviewed-on: https://cl.tvl.fyi/c/depot/+/3117 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-15 r/2865 fix(bin): correct outdated attribute pathssterni2-11/+7
* Mostly stale attribute specifications due to the third_party -> third_party.nixpkgs migration. * kontemplate is ops.kontemplate. * tazjin's rebuilder seems to no longer exist. Change-Id: Ic97468f9e73cb0ba7df740c3807ca011e385e357 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3550 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-15 r/2864 feat(bin): add nint to __dispatch.shsterni2-0/+4
Change-Id: I0e52a0e87ae15901aaabcc7196d2cd76a8b143aa Reviewed-on: https://cl.tvl.fyi/c/depot/+/3549 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-15 r/2863 feat(web/template): Link to CI in TVL page footersVincent Ambo1-0/+2
Change-Id: I53229374605774f34d3cfda2b071f65b79227a24 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3555 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-15 r/2862 refactor(nix/nint): use stdout().write_all() over print!()sterni1-3/+3
Change-Id: Ic372442c5156f1e13fea28c1b3e57d64e89ba49b Reviewed-on: https://cl.tvl.fyi/c/depot/+/3546 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-15 r/2861 refactor(nix/readTree): remove unused assertMsg functionsterni1-5/+0
Change-Id: I4048a59ca3daeb541a7294ce72eebe32982d852c Reviewed-on: https://cl.tvl.fyi/c/depot/+/3553 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-15 r/2860 feat(tazjin/tverskoy): Enable adbVincent Ambo1-1/+2
Change-Id: I088d2b0526f10d848da56d8192e93b79d6297746 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3539 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-15 r/2859 feat(users/sterni/emacs): enable rust modesterni2-0/+2
Change-Id: I49c8f5c0c18ac7664f5f120ad23a55c3bc19bd5b Reviewed-on: https://cl.tvl.fyi/c/depot/+/3545 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-14 r/2858 test(nix/utils): unit test path predicatessterni8-0/+69
Change-Id: I85f38ef8f284c4accd1ea744f61358622785358d Reviewed-on: https://cl.tvl.fyi/c/depot/+/3534 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-09-14 r/2857 feat(nix/runTestsuite): rebuild if test case list changessterni1-2/+5
If the result of the assertions changes for a successful test suite (this happens if tests are reworded, added or removed), this makes sure the no-op derivation is rebuilt. This makes sure that test suites show up in buildkite on ocassions other than channel bumps, since they are only added to the job list if their `outPath` is missing nowadays (see cl/3427). Change-Id: Ia1050cca5eeed8b7da84c40f6154b40760a3047f Reviewed-on: https://cl.tvl.fyi/c/depot/+/3536 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-09-14 r/2856 feat(nix/tag): add functions for querying a tag's name and valuesterni1-1/+14
Both are just trivial wrappers around assertIsTag to make these lookups more ergonomic. This also allows us to demote assertIsTag to an implemtation detail. Change-Id: Ib6ba2a858f4839354a57b660042b418976c4b1d9 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3541 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-09-14 r/2855 docs(nix/tag): correct comment for assertIsTagsterni1-1/+1
Change-Id: Id54cc3a1d7c83c6a94b9446c76b00ca2ce1ab48e Reviewed-on: https://cl.tvl.fyi/c/depot/+/3540 Tested-by: BuildkiteCI Reviewed-by: Profpatsch <mail@profpatsch.de>
2021-09-12 r/2854 feat(sterni/mblog): convert apple note mime msgs to htmlsterni5-0/+250
For now mblog only contains the mnote-html executable which takes a mime message from a maildir and prints the equivalent HTML fragment to stdout. It is intended to work with the mblaze(7) utilities, i. e. mnote-html resolves all `object` tags to proper `img` inclusions with the correct filename, so mshow(1)'s -x version can supply the needed image files. A note created using Apple's Notes app (tested with the iOS version) can be converted in a viewable HTML file like this: $ mnote-html path/to/msg > fragment.html $ mshow -x path/to/msg $ cat <(echo "<!DOCTYPE html>") fragment.html > document.html $ xdg-open document.html Note that only the limited feature set of Apple Notes when using the IMAP backend is supported. The iCloud-based one has more (quite neat) features, but its notes can only accessed via an internal API as far as I know. This CLI is a bit impractical due to the big startup overhead of loading the lisp image. mblog should be become a fully fletched static site generator in the future, but this is a good starting point and providing the mnote-html tool is certainly useful. Change-Id: Iee6d1558e939b932da1e70ca2d2ae75638d855df Reviewed-on: https://cl.tvl.fyi/c/depot/+/3271 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-12 r/2853 feat(3p/lisp/mime4cl): search for first (default) mime text partsterni2-0/+25
Adds a simple generic function find-mime-text-part which returns the first suitable text/* part in any MIME part it is given. Has no meaningful alternatives handling at the moment: It will pick the first text part and doesn't allow specifying a preference. Change-Id: Id9b113b3ef3ca1a575ce8f3582a4f85e30edfb43 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3379 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-12 r/2852 fix(nix/buildLisp): preserve argv[0] in buildLisp.programsterni1-7/+6
I *thought* I was being clever with the (cdr (member …)) call, but somehow completely forgot that *posix-argv* and *command-line-argument-list* are equivalent to argv, so they also contain the program name as the first element. Dropping that made argument parsing completely break down, so let's revert back to the older solution which works quite well. Change-Id: If7d3321cda0ca512bc8c23b6541ce390b81a3e24 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3538 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in>
2021-09-12 r/2851 feat(users/sterni/nix/string): very simple printf implementationsterni2-0/+45
This is mostly to yet another silly idea which turns out to be possible. This may be actually useful should I implement more sophisticated format specifiers like "%xd" or "%f". Change-Id: Ia56cd6f5793a09fe5e19c91a8e8f9098f3244d57 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3537 Tested-by: BuildkiteCI Reviewed-by: sterni <sternenseemann@systemli.org>
2021-09-11 r/2850 refactor(grfn/xanthous): avoid unnecessary recompilationsterni3-61/+3
hpack is a bit dumb when generating the list of modules for a cabal file's component if multiple of them live in the same directory. Specifically it seems to assume that all modules in the source-dirs of a particular component are also necessary for its compilation. This is quite bad in the case of xanthous since both library and executable have source-dirs: src, so all modules will be compiled twice: Once for the library and then again for the executable despite it depending on the library (actually 4 times in total since we need to build a unprofiled and profiled object for each module…). To fix this we just move Main.hs into its own directory and change the executable's source-dirs, so hpack doesn't get confused anymore. Since all components now have their own source-dirs, unnecessary redundant compilation should be down to 0. The diff of the cabal file shows quite nicely how many module recompilation we've gotten rid of. Change-Id: I2df4fab9b0299b3a2b5d3005508c79b2d9796039 Reviewed-on: https://cl.tvl.fyi/c/depot/+/3533 Tested-by: BuildkiteCI Reviewed-by: tazjin <mail@tazj.in> Reviewed-by: grfn <grfn@gws.fyi>
generated by cgit-pink 1.4.1 (git 2.44.2) at 2024-11-05T04·10+0000