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

[DAT-3942] Set up files needed for gradle #2

Open
wants to merge 2 commits into
base: upgrade-to-mx9
Choose a base branch
from

Conversation

pijuskri
Copy link
Collaborator

@pijuskri pijuskri commented Feb 4, 2025

No description provided.

@@ -0,0 +1 @@
This module allows you to execute OQL queries from a microflow. Snippet to test & export OQL queries, add parameters with actions, execute query returning entities, execute query & return a CSV, etc.
Copy link

Choose a reason for hiding this comment

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

Let's also add a release note to marketplace/release-notes


## Install Mendix

First determine the correct version using `./gradlew modelerVersion`.
Copy link

Choose a reason for hiding this comment

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

File permissions for gradlew are -rw-r--r--@, so I cannot run it out of the box.

When I do, I get ./gradlew: /bin/sh^M: bad interpreter: No such file or directory, so I guess we should have different line end characters

When I fix that, I get > Could not get unknown property 'publishModuleToMarketplacee' for root project 'OQLModule' of type org.gradle.api.Project, but I'll add that as a separate comment

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I get different permissions when running ls -l on wsl. Ill try to force execution rights to check if this works

copyDirectDependenciesToUserlib.mustRunAfter copyJarsToUserlib
}

afterReleaseBuild.dependsOn publishModuleToMarketplacee
Copy link

Choose a reason for hiding this comment

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

let's fix the task name

environment 'MX_INSTALL_PATH',mxInstallPath
}

tasks.register('copyDirectDependenciesToUserlib', Copy) {
Copy link

Choose a reason for hiding this comment

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

This one is named copyToUserlib in DatabaseReplication. We should better use similar names for tasks

Copy link
Collaborator Author

@pijuskri pijuskri Feb 13, 2025

Choose a reason for hiding this comment

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

I believe the issue here is db replication. There is already a task called copyJarsToUserlib from the publish plugin, which is would make it really confusing telling what is the difference from copyToUserlib.

Copy link

Choose a reason for hiding this comment

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

Makes sense. We'll have to fix db replication then.

group "Build"
description "Copies the direct dependencies to the userlib directory."

dependsOn clean
Copy link

@dahfjkg dahfjkg Feb 12, 2025

Choose a reason for hiding this comment

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

You can also see some logic related to the .migration file in database replication. Than one was used to migrate userlib dependencies to newly introduced managed dependencies. I think it should still be present in 9.18.4

# This will upgrade the app to the version of Mendix corresponding to APPSTUDIO build number and run tests
# Required environment:
# - APPSTUDIO;
version: "2.2"
Copy link

Choose a reason for hiding this comment

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

docker will complain that version is rudimentary

@@ -0,0 +1,13 @@
#!/bin/sh
Copy link

Choose a reason for hiding this comment

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

this file needs execution permissions

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.

2 participants