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

add koji tag location support #4

Merged
merged 5 commits into from
Aug 5, 2020
Merged

Conversation

TristanCacqueray
Copy link
Collaborator

This change enables comparing koji tag using a new syntax:
tag@kojihub

This change enables comparing koji tag using a new syntax:
  `tag`@`kojihub`
@juhp
Copy link
Owner

juhp commented Jul 30, 2020

Okay, cool, I just noticed this and will try to review this tomorrow.

I have pondered before about syntax for interesting new sources, eg ssh (but wondering if abc@xyz may be too generic, though not sure what would be better: maybe using (pseudo)-uri's like koji://tag@hub perhaps?)
Anyway we can think a little on this before releasing it.

@TristanCacqueray
Copy link
Collaborator Author

Oh right, I forgot to mention I wasn't a big fan of the tag@hub syntax :-)

The issue with using a koji:// scheme is that it doesn't look great for fully qualified hub, e.g.:
koji://sf-3.5-el7@http://koji.softwarefactory-project.io/kojihub

Though I can't think of a better solution. Let me do the change.

@juhp
Copy link
Owner

juhp commented Jul 30, 2020

The issue with using a koji:// scheme is that it doesn't look great for fully qualified hub, e.g.:
koji://sf-3.5-el7@http://koji.softwarefactory-project.io/kojihub

Hmm yea, good point - never seen url in uri before...

Guess it is not worse than xml embedded in xml, lol (Me looks into Koji rpc' eyes... ;o)

@TristanCacqueray
Copy link
Collaborator Author

Another solution would be a simpler koji: or koji= prefix.

src/Main.hs Outdated Show resolved Hide resolved
@juhp
Copy link
Owner

juhp commented Aug 4, 2020

Basically looks good to me: though I haven't tried it.

The issue with using a koji:// scheme is that it doesn't look great for fully qualified hub, e.g.:
koji://sf-3.5-el7@http://koji.softwarefactory-project.io/kojihub

How about koji://sf-3.5-el7@koji.softwarefactory-project.io/kojihub? Could that be sufficient?

Or possibly koji://koji.softwarefactory-project.io/kojihub#sf-3.5-el7 ?? Not sure which uri scheme is most natural.

stack.yaml Show resolved Hide resolved
@TristanCacqueray
Copy link
Collaborator Author

Removing the koji http scheme can be an issue too when the service is only http or https. I think that's quite a special case, and koji://tag-name@scheme://url looks like a good compromise to me.

@juhp juhp merged commit a874ac3 into juhp:master Aug 5, 2020
@juhp
Copy link
Owner

juhp commented Aug 5, 2020

Thanks, looking forward to play with this later 👍

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.

None yet

2 participants