Skip to content
This repository has been archived by the owner on May 18, 2019. It is now read-only.

Results for name with subspec are empty #65

Closed
segiddins opened this issue Nov 6, 2014 · 19 comments · Fixed by #74
Closed

Results for name with subspec are empty #65

segiddins opened this issue Nov 6, 2014 · 19 comments · Fixed by #74

Comments

@segiddins
Copy link
Member

See http://cocoapods.org/?q=RestKit%2FCoreData

@floere I'd love to start by at least splitting on the /, but maybe the subspec names should also be taken into account when searching?

@floere
Copy link
Member

floere commented Nov 6, 2014

Go ahead.
And Kyle has included the subspec names with a recent PR.

@floere
Copy link
Member

floere commented Nov 6, 2014

Hmm, maybe I misunderstood. You mean that the expression A/B should have special significance?

@segiddins
Copy link
Member Author

Yes, as A/B means

spec: 'A',
subspec: 'B',

@floere
Copy link
Member

floere commented Nov 6, 2014

I'm not sure whether the subspecs should have special significance - I don't know the domain well enough. Ping others?

@floere
Copy link
Member

floere commented Nov 6, 2014

@segiddins But I'd be definitely happy if you split on '/', for a start :)

@segiddins
Copy link
Member Author

Where should I look to be making that sort of change?

-Samuel E. Giddins

On Nov 5, 2014, at 11:34 PM, Florian R. Hanke notifications@github.com wrote:

@segiddins But I'd be definitely happy if you split on '/', for a start :)


Reply to this email directly or view it on GitHub.

@floere
Copy link
Member

floere commented Nov 6, 2014

In two places (and maybe remove the duplication):
https://github.com/CocoaPods/search.cocoapods.org/blob/trunk-database-rewrite/lib/search.rb#L120
https://github.com/CocoaPods/search.cocoapods.org/blob/trunk-database-rewrite/lib/search.rb#L149

To run the tests you need to load spec/trunk.dump into a trunk_cocoapods_org_test database (psql trunk_cocoapods_org_test < spec/trunk.dump), then run bundle exec rake.

@segiddins
Copy link
Member Author

@floere after thinking it through, I believe the correct approach would be to make the recursive subspecs searchable

@floere
Copy link
Member

floere commented Nov 6, 2014

As a separate category? So if you search for CoreData, then RestKit should be found via CoreData under category subspecs?

@segiddins
Copy link
Member Author

Yes, but shouldn't it also match the name, but with a lower weight than matching the root spec's name?

@floere
Copy link
Member

floere commented Nov 6, 2014

Not sure I understand fully.

Can we make an example?

(
        "name"=>"AFNetworking",
        "version"=>"2.3.1",
        "license"=>"MIT",
        "summary"=>"A delightful iOS and OS X networking framework.",
        "homepage"=>"https://github.com/AFNetworking/AFNetworking",
        "social_media_url"=>"https://twitter.com/AFNetworking",
        "authors"=>{"Mattt Thompson"=>"m@mattt.me"},
        "source"=>{
          "git"=>"https://github.com/AFNetworking/AFNetworking.git",
          "tag"=>"2.3.1",
          "submodules"=>true},
          "requires_arc"=>true,
          "platforms"=>{
            "ios"=>"6.0",
            "osx"=>"10.8"
          },
          "public_header_files"=>"AFNetworking/*.h",
          "source_files"=>"AFNetworking/AFNetworking.h",
          "subspecs"=>[
            {
              "name"=>"Serialization",
              "source_files"=>"AFNetworking/AFURL{Request,Response}Serialization.{h,m}",
              "ios"=>{
                "frameworks"=>[
                  "MobileCoreServices",
                  "CoreGraphics"
                ]
              },
              "osx"=>{
                "frameworks"=>"CoreServices"
              }
            },
            {
              "name"=>"Security",
              "source_files"=>"AFNetworking/AFSecurityPolicy.{h,m}",
              "frameworks"=>"Security"}, {"name"=>"Reachability",
              "source_files"=>"AFNetworking/AFNetworkReachabilityManager.{h,m}",
              "frameworks"=>"SystemConfiguration"
            },
            {
              "name"=>"NSURLConnection",
              "dependencies"=>{
                "AFNetworking/Serialization"=>[],
                "AFNetworking/Reachability"=>[],
                "AFNetworking/Security"=>[]
              },
              "source_files"=>[
                "AFNetworking/AFURLConnectionOperation.{h,m}",
                "AFNetworking/AFHTTPRequestOperation.{h,m}",
                "AFNetworking/AFHTTPRequestOperationManager.{h,m}"
              ]
            },
            {
              "name"=>"NSURLSession",
              "dependencies"=>{
                "AFNetworking/Serialization"=>[],
                "AFNetworking/Reachability"=>[],
                "AFNetworking/Security"=>[]
              },
              "source_files"=>[
                "AFNetworking/AFURLSessionManager.{h,m}",
                "AFNetworking/AFHTTPSessionManager.{h,m}"
              ]
            },
            {
              "name"=>"UIKit",
              "platforms"=>{
                "ios"=>"6.0"
              },
              "dependencies"=>{
                "AFNetworking/NSURLConnection"=>[],
                "AFNetworking/NSURLSession"=>[]
              },
              "ios"=>{
                "public_header_files"=>"UIKit+AFNetworking/*.h",
                "source_files"=>"UIKit+AFNetworking"
              },
              "osx"=>{
                "source_files"=>""
              }
            }
          ]
        )

Should this pod be found when entering Serialization or Security?

@segiddins
Copy link
Member Author

Yes, imho

-Samuel E. Giddins

On Nov 6, 2014, at 1:23 PM, Florian R. Hanke notifications@github.com wrote:

Not sure I understand fully.

Can we make an example?

(
"name"=>"AFNetworking",
"version"=>"2.3.1",
"license"=>"MIT",
"summary"=>"A delightful iOS and OS X networking framework.",
"homepage"=>"https://github.com/AFNetworking/AFNetworking",
"social_media_url"=>"https://twitter.com/AFNetworking",
"authors"=>{"Mattt Thompson"=>"m@mattt.me"},
"source"=>{
"git"=>"https://github.com/AFNetworking/AFNetworking.git",
"tag"=>"2.3.1",
"submodules"=>true},
"requires_arc"=>true,
"platforms"=>{
"ios"=>"6.0",
"osx"=>"10.8"
},
"public_header_files"=>"AFNetworking/.h",
"source_files"=>"AFNetworking/AFNetworking.h",
"subspecs"=>[
{
"name"=>"Serialization",
"source_files"=>"AFNetworking/AFURL{Request,Response}Serialization.{h,m}",
"ios"=>{
"frameworks"=>[
"MobileCoreServices",
"CoreGraphics"
]
},
"osx"=>{
"frameworks"=>"CoreServices"
}
},
{
"name"=>"Security",
"source_files"=>"AFNetworking/AFSecurityPolicy.{h,m}",
"frameworks"=>"Security"}, {"name"=>"Reachability",
"source_files"=>"AFNetworking/AFNetworkReachabilityManager.{h,m}",
"frameworks"=>"SystemConfiguration"
},
{
"name"=>"NSURLConnection",
"dependencies"=>{
"AFNetworking/Serialization"=>[],
"AFNetworking/Reachability"=>[],
"AFNetworking/Security"=>[]
},
"source_files"=>[
"AFNetworking/AFURLConnectionOperation.{h,m}",
"AFNetworking/AFHTTPRequestOperation.{h,m}",
"AFNetworking/AFHTTPRequestOperationManager.{h,m}"
]
},
{
"name"=>"NSURLSession",
"dependencies"=>{
"AFNetworking/Serialization"=>[],
"AFNetworking/Reachability"=>[],
"AFNetworking/Security"=>[]
},
"source_files"=>[
"AFNetworking/AFURLSessionManager.{h,m}",
"AFNetworking/AFHTTPSessionManager.{h,m}"
]
},
{
"name"=>"UIKit",
"platforms"=>{
"ios"=>"6.0"
},
"dependencies"=>{
"AFNetworking/NSURLConnection"=>[],
"AFNetworking/NSURLSession"=>[]
},
"ios"=>{
"public_header_files"=>"UIKit+AFNetworking/
.h",
"source_files"=>"UIKit+AFNetworking"
},
"osx"=>{
"source_files"=>""
}
}
]
)
Should this pod be found when entering Serialization or Security?


Reply to this email directly or view it on GitHub.

@segiddins
Copy link
Member Author

I'm my opinion, yes.

@floere
Copy link
Member

floere commented Nov 6, 2014

Under category "subspecs"? I'm just not sure I understand the use case fully – a user wants to find AFNetworking, but does not know the name. But they remember that a subspec of it was "Serialization", so they enter that. Does that make sense?

@floere
Copy link
Member

floere commented Dec 14, 2014

Should we make this: subspec:CoreData, and it will find RestKit?

@floere
Copy link
Member

floere commented Dec 19, 2014

Ping

@floere
Copy link
Member

floere commented Dec 22, 2014

Can I close this? We're not making any progress.

@segiddins
Copy link
Member Author

I will make progress, I promise. I just dropped the ball on this

-Samuel E. Giddins

On Dec 22, 2014, at 7:16 AM, Florian R. Hanke notifications@github.com wrote:

Can I close this? We're not making any progress.


Reply to this email directly or view it on GitHub.

@floere
Copy link
Member

floere commented Dec 22, 2014

No worries! I'd just like to reduce the issue count. It's not urgent :)

On Mon, Dec 22, 2014 at 11:20 AM, Samuel E. Giddins
notifications@github.com wrote:

I will make progress, I promise. I just dropped the ball on this
-Samuel E. Giddins
On Dec 22, 2014, at 7:16 AM, Florian R. Hanke notifications@github.com wrote:
Can I close this? We're not making any progress.

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub:
#65 (comment)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants