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

Cannot find symbol GenericEditorArea.TEMPLATE_TYPE_KEY #10

Open
lazear opened this issue Nov 17, 2022 · 4 comments
Open

Cannot find symbol GenericEditorArea.TEMPLATE_TYPE_KEY #10

lazear opened this issue Nov 17, 2022 · 4 comments

Comments

@lazear
Copy link

lazear commented Nov 17, 2022

Hi,

I'm trying to build (on Windows, using amazon corretto jdk8) the latest version cloned from GitHub - having a couple issues:

Github\datawarrior\src\com\actelion\research\datawarrior\DataWarrior.java:285:60
java: cannot find symbol
  symbol:   variable TEMPLATE_TYPE_KEY
  location: class com.actelion.research.gui.editor.GenericEditorArea
Github\datawarrior\src\com\actelion\research\datawarrior\DataWarrior.java:286:60
java: cannot find symbol
  symbol:   variable TEMPLATE_TYPE_REACTION_QUERIES
  location: class com.actelion.research.gui.editor.GenericEditorArea
Github\datawarrior\src\com\actelion\research\datawarrior\DataWarrior.java:295:38
java: cannot find symbol
  symbol:   method setReactionQueryTemplates(java.lang.String[][])
  location: class com.actelion.research.gui.editor.GenericEditorArea

git blame says this was introduced ~9 days ago - perhaps the openchemlib.jar file has been updated locally for you, but not checked into git? 1d55833

Also getting 3 errors related to the following:

Github\datawarrior\src\com\actelion\research\datawarrior\task\chem\DETaskAdd3DCoordinates.java:759:36
java: non-static method neutralizeChargedMolecule(com.actelion.research.chem.StereoMolecule) cannot be referenced from a static context

Is there a specific git commit I should be trying to build from?

@lazear
Copy link
Author

lazear commented Nov 17, 2022

I can build successfully on f83b589

@thsa
Copy link
Owner

thsa commented Nov 18, 2022 via email

@lazear
Copy link
Author

lazear commented Nov 18, 2022

Thanks!

I've just added support for AWS Athena as a SQL database provider in our fork: https://github.com/belharra-tx/datawarrior/tree/athena

Unfortunately, this took a couple hours of debugging because the AWS Athena driver comes bundled with log4j v2.12.1 and the opsin.jar library provides v2.18 ... there is a class method present in v2.12 that was deleted in v2.18 (!! - apparently log4j is not using semantic versioning??). I had to build my own version of opsin with log4j v2.12.1 to resolve this issue - seems like this is something Maven could help prevent (I am a total Java neophyte, so not sure if that's how Java deps work)?

I saw a relevant issue on the tracker: #6 - would you like some help with this? I'm wondering if it's possible for you to have github projects with your forks of the libraries you have manually edited. And also if you are interested in getting some kind of CI/CD or release/tag version set up on GitHub?

I think we will be using DataWarrior internally so I can probably dedicate a few hours to helping out with these issues if you are interested!

@mjw99
Copy link

mjw99 commented Feb 27, 2023

@lazear BTW, I have an experimental fork of a mavenised form of datawarrior here; please see the instructions in that fork's readme for compiling.

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

No branches or pull requests

3 participants