Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SOLR-15178 Non-existent dependency listed in solr-core #2415

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

bszabo97
Copy link

Description

Solr-core has a dependency, org.apache.solr:server, which fails to download.
For testing I created a test project here: https://github.com/bszabo97/solr_master_dep_test
If I run the command gradle -q dependencies --configuration solrCore the dependency org.apache.solr:server shows up and fails, though if I run gradle -q dependencies --configuration solrCore8 it doesn't show up at all.

Solution

Adding the missing artifact.

Tests

Unit tests

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
  • I have developed this patch against the master branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Ref Guide (for Solr changes only).

dweiss and others added 23 commits March 10, 2021 12:43
* Basic cleanup of README
* Remove lucene DOAP file
* Change gradle rootProject.name to solr-root
* Remove badge and some more Lucene mentions. Added a link to Lucene webpage.
…s ensures consistent use patterns across JVMs.
This allows us to re-enable the checkLocalJavadocLinksSite task, and in theory will point to the 'correct' URLs once 9.0 comes out, but there are longer term fixes still needed (see SOLR-15262)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants