Skip to content
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

[projmgr] MVP prototype initial commit #1

Draft
wants to merge 30 commits into
base: main
Choose a base branch
from
Draft

Conversation

brondani
Copy link
Owner

@brondani brondani commented Oct 4, 2021

No description provided.

@brondani brondani force-pushed the mvp-prototype branch 17 times, most recently from 3e9b419 to 6ef6ec7 Compare October 5, 2021 11:25
@brondani brondani marked this pull request as draft October 5, 2021 11:31
@0Grit
Copy link

0Grit commented Oct 18, 2021

Anything but python please.

```

The proposed component name syntax is:
`[Cvendor::] Cclass [&Cbundle] :Cgroup [:Csub] [&Cvariant] [@[>=]Cversion]`
Copy link

@JonatanAntoni JonatanAntoni Nov 9, 2021

Choose a reason for hiding this comment

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

The >= operator has the drawback of potentially silently select incompatible versions, for example @>=1.0 could select version 2.0 which is considered a major (potentially incompatible) update. Other tools make use of the ~= operator to select compatible versions, see Python for instance. We could allow @~=1.1 which means version range [1.1.0, 2.0.0).

Section Name Conventions introduced
Copy link

@JonatanAntoni JonatanAntoni left a comment

Choose a reason for hiding this comment

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

Please don't add binaries into the repository. You could create a release instead and attache the binaries there.

@brondani
Copy link
Owner Author

Please don't add binaries into the repository. You could create a release instead and attache the binaries there.

This is a temporary solution agreed with @ReinhardKeil and @jkrech. The sources for these binaries are not yet public.

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.

6 participants