-
Notifications
You must be signed in to change notification settings - Fork 2.4k
SE-0292 revision 2 #1301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SE-0292 revision 2 #1301
Conversation
I didn't see anything that describes the client behavior when the client is configured with a registry URL (not a scope-specific one) and the client encounters a dependency in a I see that the If the registry doesn't know about the URL then does the client fetch the package by URL? |
A couple other questions related to my last comment:
|
@mmarston Thanks for all of your feedback, by the way! Some answers to your recent questions below:
Yes, that's the intention. The exact behavior is an implementation behavior on the client's end, but my understanding is that URL-based package dependencies will continue to coexist alongside registry identifiers, at least initially.
This is another implementation detail, but my intention is for only the default / unscoped registry to be consulted.
This proposal doesn't cover that, but I'd imagine that this behavior could be controlled by a command-line flag.
No. Any changes to the package source code would break the checksums computed by the client. |
This change was made at the request of the Swift core team ahead of second review
|
||
```regexp | ||
\A[a-zA-Z\d](?:[a-zA-Z\d]|-(?=[a-zA-Z\d])){0,39}\z | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @neonichu @abertelrud @yim-lee: are these general enough?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think 39 chars limit is too low but open to see what the community thinks.
|
||
```regexp | ||
\A\p{XID_Start}\p{XID_Continue}{0,127}\z | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @neonichu @abertelrud @yim-lee: are these general enough?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I might be wrong, but I believe this is the same as what SwiftPM uses for package names today?
I think scope and name should support the same characters but given the thread on removing name from the manifest I am ok with leaving this as is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SwiftPM places no restrictions on package names, AFAIK.
This change was made at the request of the Swift core team ahead of second review
…tion' This change was made at the request of the Swift core team ahead of second review
This change was made at the request of the Swift core team ahead of second review
Changes from the original proposal:
@mona/LinkedList
mona.LinkedList
identity scheme to replace package URLs in dependency declarationspackage-registry set|unset <url>
subcommands for managing local and global package registry configurationarchive
subcommand toarchive-source
GET /identifiers{?url}
endpoint