See #2157839-4: Copy drupal-4.2-solr-XX schema and solrconfig changes to the module: there are some typos, left-overs from accidental commits and wrong version numbers present in the config files tagged with 4.2 These should be fixed.
(I don't think separate issues are necessary, since these are all straight-forward little fixes.)
I have also included two comment style changes in solrcore.properties so they conform to the Drupal coding standards. Since this is our own file, there shouldn't be any problems regarding compatibility to the default Solr configs.

CommentFileSizeAuthor
#1 2158939-1--small_fixes.patch6.08 KBdrunken monkey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey’s picture

Status: Active » Needs review
FileSize
6.08 KB

Oh, and the "salves" typo was also introduced by us and isn't in the default config files.
Patch attached.

Nick_vh’s picture

the name is actually correct in the 4.2 tag? This patch won't apply

drunken monkey’s picture

I just checked, and it applies fine for me with a freshly updated master branch of this module. What error message do you receive when applying it, for what file does it fail?

drunken monkey’s picture

Status: Needs review » Fixed

Got the clearance from Nick, so: committed.
Also created a new 4.2 tag for the changes. I guess it might be a bit confusing now for others, since there are two "4.2" tags, but I don't think more than a few people are really looking that closely at this module.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.