Details

      Description

      The /clients directory is a bit of a mess. The only actively maintained client SolrJ is actually in the /dist directory! The other clients that used to be in here, /php and /javascript (I think!) have been moved. The only one is /ruby, and it isn't actively maintained the way other ruby clients are.

      I'd recommend just removing the /clients directory since it's very confusing to a new user who would logically go here to find clients, and only find a ruby one! It would also let us slim down the size of the download.

      Alterntively if we want the /clients directory, then lets copy over the solrj lib to this dir instead of /dist

      I am happy to submit a patch if this makes sense.

      1. SOLR-2607.patch
        1 kB
        Jan Høydahl

        Activity

        Hide
        Jan Høydahl added a comment -

        +1 I saw this earlier but just thought about it again when reviewing the 3.5.0 RC. I'd say get rid of /client altogether

        Show
        Jan Høydahl added a comment - +1 I saw this earlier but just thought about it again when reviewing the 3.5.0 RC. I'd say get rid of /client altogether
        Hide
        Erik Hatcher added a comment -

        I'll take this one... since I'm the culprit littering /clients now. I'm going to move solr-ruby and flare over to my github account. solr-ruby will be entirely deprecated in favor of RSolr, and I'll be moving the remaining goodies from solr-ruby (Mapper and Indexer, both very handy features not found elsewhere) into RSolr.

        I'll aim to tackle this before years-end over holiday breaks.

        Show
        Erik Hatcher added a comment - I'll take this one... since I'm the culprit littering /clients now. I'm going to move solr-ruby and flare over to my github account. solr-ruby will be entirely deprecated in favor of RSolr, and I'll be moving the remaining goodies from solr-ruby (Mapper and Indexer, both very handy features not found elsewhere) into RSolr. I'll aim to tackle this before years-end over holiday breaks.
        Hide
        Erik Hatcher added a comment -

        I went ahead and took care of this on trunk. svn exported to https://github.com/erikhatcher/solr-ruby-flare.

        RIP solr-ruby and flare. Long live Flare!

        For future reference, if you're using solr-ruby, consider switching to RSolr - http://rubygems.org/gems/rsolr

        Flare was the precursor prototype that morphed into the popular Blacklight framework - http://projectblacklight.org

        Show
        Erik Hatcher added a comment - I went ahead and took care of this on trunk. svn exported to https://github.com/erikhatcher/solr-ruby-flare . RIP solr-ruby and flare. Long live Flare! For future reference, if you're using solr-ruby, consider switching to RSolr - http://rubygems.org/gems/rsolr Flare was the precursor prototype that morphed into the popular Blacklight framework - http://projectblacklight.org
        Hide
        Erik Hatcher added a comment -

        going to re-open... we may want to also remove client/ fully? It's nice to have the pointer to the wiki though. Thoughts?

        Show
        Erik Hatcher added a comment - going to re-open... we may want to also remove client/ fully? It's nice to have the pointer to the wiki though. Thoughts?
        Hide
        Jan Høydahl added a comment -

        I'd say kill client/ and add a paragraph in README.TXT about clients with a link to Wiki.

        Show
        Jan Høydahl added a comment - I'd say kill client/ and add a paragraph in README.TXT about clients with a link to Wiki.
        Hide
        Jan Høydahl added a comment -

        Attached patch removes clients folder and adds two lines to README. Waddaya think, time to resolve this issue?

        Show
        Jan Høydahl added a comment - Attached patch removes clients folder and adds two lines to README. Waddaya think, time to resolve this issue?
        Hide
        Erik Hatcher added a comment -

        Thanks Jan, committed.

        Show
        Erik Hatcher added a comment - Thanks Jan, committed.

          People

          • Assignee:
            Erik Hatcher
            Reporter:
            Eric Pugh
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development