Does Not Reliably Detect Origin / Remotes about circleci-config-generator

[eddiewebb]

Thank you for the awesomely detailed and thoughtful issue submission! I fully understand the need to properly detect origin in the case of multiple upstream repos (a relatively common pattern)

Would you be willing to submit this as PR @GuyPaddock ?

[woodardj]

Would be nice to get this fixed.

[eddiewebb]

@GuyPaddock, @woodardj , please try the referenced changes in https://github.com/CircleCI-Public/circleci-config-generator/blob/issue-5-issue-1/cci-config-generator.sh and confirm this works for multiple remotes (will always use origin).

[woodardj]

Negative.

Here's my remote -v list:

Rockhopper:<<project-name>> woodardj[master]$ git remote -v
heroku	https://git.heroku.com/<<project-name>>.git (fetch)
heroku	https://git.heroku.com/<<project-name>>.git (push)
origin	[email protected]:woodardj/<<project-name>>.git (fetch)
origin	[email protected]:woodardj/<<project-name>>.git (push)

Here's what I see when I run the new script:

Rockhopper:<<project-name>> woodardj[master]$ ./cci-config-generator.sh 
Gathering info please wait...

 !  Invalid path.
 !  Syntax is: [email protected]:<app>.git where <app> is your app's name.
 !  
 !  SSH Key Fingerprint: <<snip>>

fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
git origin references `` hosted by heroku, which will be used as source.
Paste your CircleCI API token here: ^C

[eddiewebb]

Just updated, what's the output for git remote get-url --push origin | perl -ne 'print $1 if /([^\/:]*\/[^\/]*?)(.git)?$/'

[woodardj]

woodardj/<<project-name>>

still seeing same behavior from script. will try again in a few hours.

[eddiewebb]

Hi @woodardj thanks for validating. Your comment above changed, so I just want to be clear. THe latest script (on the branch above) should now support multiple remotes, always choosing origin. The relevant portions are:

vcs_provider="$(git remote get-url --push origin | perl -ne 'print $1 if /([A-Za-z]*)\.(com|org)/')"
project="$(git remote get-url --push origin | perl -ne 'print $1 if /([^\/:]*\/[^\/]*?)(.git)?$/')"

So if the script doesn't run, please execute those and then grab their values (i.e. echo "$project at $vcs_provider")

I'm testing locally against various patterns noted in all 3 open issues, want to be sure I haven't omitted something!

[woodardj]

Looks like it's all good now — this script was super smooth and easy once it worked! Thanks for putting it together! Now I've just gotta figure out why one of my repos is failing.

[eddiewebb]

Fixed in #9