Skip to content

Conversation

c-vandendyck-kbr
Copy link
Contributor

Update ENV:

  • solrwayback 5.1
  • tomcat 9
  • solr 9

Also update RUN commands in order to call the good folder and the -c option

@thomasegense
Copy link
Contributor

Thanks. I will look into it tomorrow!

@c-vandendyck-kbr
Copy link
Contributor Author

Hi @thomasegense !

And sorry it took me so much time ;-)

@VictorHarbo VictorHarbo self-requested a review July 19, 2024 08:19
Copy link
Collaborator

@VictorHarbo VictorHarbo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is indeed working as it should. Thanks a lot! The Docker file contains documentation on how to start the service in the lines 1-22. I would like the following changes here:

  • Could you please update exports in line 10-12 to match the ENV set in the file.
  • In line 13 add -c to the command.
  • in line 14 change apache-tomcat to tomcat, as thats the name of the directory now.

@VictorHarbo
Copy link
Collaborator

Ps. I've tested the Dockerfile on a Macbook running an M1 processor and it works flawlessly.

@thomasegense
Copy link
Contributor

@VictorHarbo When you accept the PR, also close this Jira: #439

@c-vandendyck-kbr
Copy link
Contributor Author

Hi @VictorHarbo !

Thanks for your feedback!
In fact, in the excitement of this first pull-request for the solrwayback project, I completely forgot about the documentation.
My mistake !

I've just corrected it ;-)

Copy link
Collaborator

@VictorHarbo VictorHarbo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Single change and then it is good to go :)

@VictorHarbo
Copy link
Collaborator

Hi @VictorHarbo !

Thanks for your feedback! In fact, in the excitement of this first pull-request for the solrwayback project, I completely forgot about the documentation. My mistake !

I've just corrected it ;-)

No worries. We love to have excited people contributing! One minor change and I'll merge it.

Copy link
Collaborator

@VictorHarbo VictorHarbo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @c-vandendyck-kbr. Everything looks good now. Congratulations on your first contribution to SolrWayback! 🎆

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.

3 participants