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

Tomcat start with Eclipse on Windows broken, local-config ignored #5999

Open
matthias-ronge opened this issue Mar 8, 2024 · 4 comments · May be fixed by #6000
Open

Tomcat start with Eclipse on Windows broken, local-config ignored #5999

matthias-ronge opened this issue Mar 8, 2024 · 4 comments · May be fixed by #6000

Comments

@matthias-ronge
Copy link
Collaborator

Describe the bug
Tomcat no longer starts from Eclipse under Windows. The reason turns out to be that the kitodo_config.properties from the config-local directory is no longer deployed, but rather the one from Kitodo/src/main/resources. Without the local path information, Tomcat will not start, not least because it can no longer find the modules directory.

To Reproduce
Steps to reproduce the behavior:

  1. Launch Tomcat from Eclipse
  2. See error

Expected behavior
Build must use kitodo_config.properties from the config-local.

Release
Current master (3.7.0-SNAPSHOT)

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: Firefox 123.0.1

Additional context
When Tomcat starts and logging was set to debug level before, you will find a log entry like:

Config - Using configuration from file:/<workspace-path>/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/kitodo/WEB-INF/classes/kitodo_config.properties

Previously, this file was a copy of config-local/kitodo_config.properties, and now it is a copy of Kitodo/src/main/resources/kitodo_config.properties.

@matthias-ronge
Copy link
Collaborator Author

Testing revealed that the error was introduced with commit 8062b8c. Undoing this change fixes it.

@henning-gerhardt
Copy link
Collaborator

henning-gerhardt commented Mar 8, 2024

The development profile is active? If not the default resources are used. Even maybe all-tests and / or selenium profile interact with the copying of files.

Edit: It is working on Linux (MacOS maybe too) with and without Idea. So it looks to me a Eclipse and / or Windows specific issue and not a issue in general. So adding the "blocking" label is a little bit overrated or?

@matthias-ronge
Copy link
Collaborator Author

Tests don’t execute when running from the command prompt. The reason is the same, the configuration does not fit:

[INFO] Running org.kitodo.imagemanagement.ImageManagementIT
[DEBUG] 2024-03-11 10:07:59.898 [main] PropertiesConfiguration - FileName set to kitodo_config.properties
[DEBUG] 2024-03-11 10:07:59.909 [main] ConfigurationUtils - ConfigurationUtils.locate(): base is null, name is kitodo_config.properties
[DEBUG] 2024-03-11 10:07:59.909 [main] DefaultFileSystem - Could not locate file kitodo_config.properties at null: no protocol: kitodo_config.properties
[WARN ] 2024-03-11 10:07:59.929 [main] Config - Loading of kitodo_config.properties failed. Trying to start with empty configuration.
org.apache.commons.configuration.ConfigurationException: Cannot locate configuration source kitodo_config.properties

It is not possible to develop quickly in this way.

@henning-gerhardt
Copy link
Collaborator

Only to be sure: the development profile is active?

But even without an active development profile the tests and integration tests should run without a locale configuration as they do it on the GitHub actions every time.

How did you can develop in the past as the commit was merged into master in December 2023? Maybe there are other changes since December 2023 on your system which now results in this errors?

Don't get me wrong as I would help you but without an Windows based system I can't and I can only ask you different things.

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

Successfully merging a pull request may close this issue.

2 participants