-
Notifications
You must be signed in to change notification settings - Fork 9
#12: Reference Server MariaDB Initial Release #31
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
base: main
Are you sure you want to change the base?
Conversation
310b9cd
to
850e063
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made some comments on things to look out for and address. Shouldn't need to spend too much time on fixing things - some are just observations.
|
||
## ENVIRNONMENT SPECIFIC NOTES | ||
|
||
The build scripts were moved to take place in a Docker container so that they would work consistently across environments. | ||
|
||
### Windows | ||
|
||
In Windows, running under a Bash shell will work, assuming you meet the above requirements. | ||
Don't forget to have Docker installed for Windows. | ||
|
||
There is an `env-default-windows` file you should rename to `.env` before running the build script. | ||
|
||
The `docker/docker-builder` file has a line commented out for Windows users, and a line that needs to be commented out. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: @patrickcoca - need to add Mac build instructions and Linux if they're different. Sometimes ARM64 builds are different from AMD64 depending on which version of Linux is being used under the hood. Debian Bullseye seems to work OK.
@@ -1,11 +1,18 @@ | |||
FROM ubuntu:20.10 | |||
FROM ubuntu:20.04 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@patrickcoca - may want to take a look at this Docker Compose file and migrate from Ubuntu to Debian. https://github.com/RESOStandards/reso-certification-utils/blob/main/DockerfileBullseye
src/main/java/org/reso/service/data/meta/MySQLFilterExpressionVisitor.java
Outdated
Show resolved
Hide resolved
src/main/java/org/reso/service/data/meta/MySQLFilterExpressionVisitor.java
Outdated
Show resolved
Hide resolved
src/main/java/org/reso/service/edmprovider/RESOedmProvider.java
Outdated
Show resolved
Hide resolved
…release-fixes Fixes and code cleaning
Closes #12.