!djTaTBQyWEPRQxrPTb:nixos.org

Nixpkgs Architecture Team

228 Members
https://github.com/nixpkgs-architecture, weekly public meetings on Wednesday 15:00-16:00 UTC at https://meet.jit.si/nixpkgs-architecture53 Servers

Load older messages


SenderMessageTime
22 Jun 2023
@archive:matrix.org@archive:matrix.org set their display name to archive.matrix.org/faq.05:26:27
23 Jun 2023
@roberthensing:matrix.orgRobert Hensing (roberth)We should really really write down migration guidelines10:23:47
@roberthensing:matrix.orgRobert Hensing (roberth) "manually" holding off migration disasters like this is a fucking drain https://github.com/NixOS/nixpkgs/pull/238331#pullrequestreview-1494880730 10:24:34
@roberthensing:matrix.orgRobert Hensing (roberth)apologies for the rude language10:24:41
@raitobezarius:matrix.orgraitobezariusI would argue that it would be a good occasion to clarify10:27:05
@raitobezarius:matrix.orgraitobezarius(1) Nixpkgs Internal Changes "release notes" (how to communicate API breakages, etc.) across our users and more10:27:20
@raitobezarius:matrix.orgraitobezarius(2) Public API vs private API promises (not necessarily a solution, but maybe have an easy way to declare from the get-go "this is private", "this is public")10:27:46
@raitobezarius:matrix.orgraitobezarius(I don't think versioning at this step makes any sense)10:28:04
@qyliss:fairydust.spaceAlyssa RossI don't see how a public vs. private distinction could make sense for Nixpkgs10:28:43
@roberthensing:matrix.orgRobert Hensing (roberth)The release notes are often quite useless with stuff like renames that are already handled by the system. Just look at the log.10:29:15
@roberthensing:matrix.orgRobert Hensing (roberth)Public: everything that's documented10:29:22
@qyliss:fairydust.spaceAlyssa RossWhat does "documented" mean in this case?10:29:49
@roberthensing:matrix.orgRobert Hensing (roberth)Private: use common sense. Don't import files except the listed ones <list>10:29:53
@piegames:matrix.org@piegames:matrix.org
In reply to @roberthensing:matrix.org
The release notes are often quite useless with stuff like renames that are already handled by the system. Just look at the log.
We could easily add the release notes editors to summarize these instead
10:29:54
@roberthensing:matrix.orgRobert Hensing (roberth)Documented in the manual10:30:01
@qyliss:fairydust.spaceAlyssa Rossah10:30:06
@qyliss:fairydust.spaceAlyssa RossI'm not sure we're ever promised that stable and unstable would be API-compatible10:32:08
@roberthensing:matrix.orgRobert Hensing (roberth)Well, about time then10:32:46
@roberthensing:matrix.orgRobert Hensing (roberth)Or are we going to break people's stuff at a whim10:32:56
@qyliss:fairydust.spaceAlyssa Rossthat's what "unstable" means10:33:04
@qyliss:fairydust.spaceAlyssa Rossand in fact, we're not breaking anything, we're just adding warnings10:33:23
@roberthensing:matrix.orgRobert Hensing (roberth)Unstable means "let's break stable for everyone who cares about unstable"?10:33:32
@roberthensing:matrix.orgRobert Hensing (roberth)that can't be right10:33:34
@roberthensing:matrix.orgRobert Hensing (roberth)if we want nixpkgs to be a drag on third party maintainers, then that's how you do it10:34:08
@qyliss:fairydust.spaceAlyssa RossI'm sympathetic to the idea that we should not add warnings to functions that are the only option on stable, but it's certainly not a widespread existing convention.10:34:10
@qyliss:fairydust.spaceAlyssa RossAnd it has downsides for Nixpkgs maintainability — it turns an atomic change into a six month long project. 10:34:35
@qyliss:fairydust.spaceAlyssa RossWhat if we backported the new functions, without a warning?10:35:00
@qyliss:fairydust.spaceAlyssa RossAnd then added the warning on unstable?10:35:07
@qyliss:fairydust.spaceAlyssa RossIt would be a totally valid backport, because it would be entirely additive.10:35:33
@roberthensing:matrix.orgRobert Hensing (roberth)That's still not atomic, so I wouldn't bother with shortening the deprecation window10:36:17

Show newer messages


Back to Room ListRoom Version: 9