depends on racket-doc
#17
Closed
opened 5 years ago by clacke
·
7 comments
Loading…
Reference in New Issue
There is no content yet.
Delete Branch '%!s(<nil>)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
It would be nice to split brag into brag-lib and brag-doc, see stchang/graph#35 for motivation.
I am willing to create the PR.
Though apparently it’s a little trickier than it looks?
Is supporting Racket 5 important for brag?
No, but making my life easy is definitely a design goal of
brag
. I have not followed this package-splitting practice for any of my other packages because it makes more work for me, and it has not presented any practical problems for users of my software. If I’m missing something, I invite correction. Otherwise I will table this issue as falling beneath the cost–benefit threshold.For most users it won't make a difference, but for Nix packaging, basing everything on racket-minimal and making the transitive dependency set as small as possible makes life much easier -- we're talking build time differences on the scale of half an hour when everything is newly bumped and no dependencies have been cached yet.
How about I make the PR providing the split, and you can look at it and see how high the maintenance burden looks? If you don't like it, I'll use plan B and synthesize a brag-lib in racket2nix instead.
OK, this is done, but could you please test it to make sure it works the way you expect.
Cool, thanks! Will have a look.
It's excellent. Thank you!