Closed yas375 closed 9 years ago
No, it's fine as it's not clear if it's a search or cocoapods.org issue. I'm guessing it may be the JS in cp.org. @orta?
On Tue, Jan 20, 2015 at 5:23 PM, Victor Ilyukevich notifications@github.com wrote:
Steps:
- Open cocoapods.org
- Enter 'confirmation' as a search query.
- You should see
MRConfirmationAlertView
in search results.- Click on its name or on the Site link. Expected: it opens https://github.com/mricketson/MRConfirmationAlertView (according to the spec) Result: it opens https://github.com/mattt/TransformerKit
P.S. I'm sorry if I choose wrong repository to open an issue in. Maybe it should be under cocoapods/cocoapods.org instead?
Reply to this email directly or view it on GitHub: https://github.com/CocoaPods/search.cocoapods.org/issues/78
Interesting, will need to look in the db for this. The site is pretty dumb, it takes whatever from the results.
Yep, looks like, the first version of the spec included that as the homepage: https://github.com/mricketson/MRConfirmationAlertView/blob/master/MRConfirmationAlertView.podspec
Later versions fixed this, but we must be grabbing that pod_version ( presumably you're doing a .first
somewhere in search )
Good job! :) I'll have to check how search gets the last version. It uses Gem::Version, but I could have implemented it wrong.
The code appears to correctly use the right version. But I can't check for this pod. What is the latest version, what the one whose spec. Is used?
@floere it should be using Pod::Version
:)
15618
has Mattt's link, it correlates to Podspec version 1.0.0. The rest don't have it.
Version 15618? (No, the version id, I guess)
@orta What is the name of version #15618?
This one is weird actually!
all 5 commits are associated with the same pod_version
.
these are the reasons why we discourage editing via PRs.
Btw: https://github.com/CocoaPods/Specs/commit/eac152fa1fdef4f305fc67cd2a6a89661f3ca16f, this change pre-dated the move to JSON specs. But it was broken, the migrator (from Ruby DSL to JSON specs in Trunk) must do the wrong with with multiple versions of the same spec.
That means we're not using the right commit.
@orta What do you mean by "these"?
Ok, got it. Thanks! @yas375 You've helped me find a tiny bug related to this issue.
well, ideally we should have a 1 to 1 ratio of commit
to version
otherwise we don't really know which is the most valid one. In theory we'd need to do a order by created_by
for a best guess.
Fixed: http://cocoapods.org/?q=confirmation Cheers!
Just a little more than an hour... You guys rock! :star2:
( Also, @kylef that makes sense )
@yas375 Thanks! I was on a bad phone connection, otherwise we may have done it in 20 minutes ;)
Btw, the problem was that I had assumed generally a 1:1 correspondence, so a random commit was taken.
@yas375 Thanks! I was on a bad phone connection, otherwise we may have done it in 20 minutes ;)
)))) :+1:
Steps:
MRConfirmationAlertView
in search results.Expected: it opens https://github.com/mricketson/MRConfirmationAlertView (according to the spec) Result: it opens https://github.com/mattt/TransformerKit
P.S. I'm sorry if I choose wrong repository to open an issue in. Maybe it should be under cocoapods/cocoapods.org instead?