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

[GR-52025] Use update center for NetBeans 14 in IGV #8290

Merged

Conversation

JaroslavTulach
Copy link
Contributor

Looks like IGV has been updated to use NetBeans-14, however the system is still configured to use update centers from NetBeans 12.5 - that's unfortunate as the latest versions of plugins aren't available. This PR aligns the IGV's base NetBeans version with its update centers.

CCing @tkrodriguez, @sdedic, @dbalek, @Ondrej-Douda

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 1, 2024
@JaroslavTulach
Copy link
Contributor Author

JaroslavTulach commented Feb 1, 2024

With this fix, I can install upcoming version of Enso4Igv plugin into IGV,

@tkrodriguez
Copy link
Member

Yes this looks good. I'll integrate it.

@JaroslavTulach
Copy link
Contributor Author

Thank you, Tom. Will you also update the binary that is used when

graal/compiler$ mx igv

please? I could change the suite.py URL, but I certainly cannot upload the new bits to lafo...

@fniephaus fniephaus changed the title Use update center for NetBeans 14 in IGV [GR-52025] Use update center for NetBeans 14 in IGV Feb 12, 2024
@tkrodriguez
Copy link
Member

Yes I will also update the binary. I was actually going to update to netbeans 20 at the same time but I'm hitting confusing problems with the dependencies and the unit tests. In particular the tests seem to have trouble loading net.java.html

    [junit] Feb 12, 2024 1:27:21 PM org.netbeans.core.startup.NbEvents notify
    [junit] WARNING: Warning - could not install some modules:
    [junit] 	Settings - The module named net.java.html was needed and not found.
    [junit] 	Settings - The module named net.java.html.json was needed and not found.

even though Settings has a clear dependence on it

<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://www.netbeans.org/ns/project/1">
    <type>org.netbeans.modules.apisupport.project</type>
    <configuration>
        <data xmlns="http://www.netbeans.org/ns/nb-module-project/3">
            <code-name-base>org.graalvm.visualizer.settings</code-name-base>
            <suite-component/>
            <module-dependencies>
                <dependency>
                    <code-name-base>net.java.html</code-name-base>
                    <build-prerequisite/>
                    <compile-dependency/>
                    <run-dependency>
                        <specification-version>1.8.1</specification-version>
                    </run-dependency>
                </dependency>
                <dependency>
                    <code-name-base>net.java.html.json</code-name-base>
                    <build-prerequisite/>
                    <compile-dependency/>
                    <run-dependency>
                        <specification-version>1.8.1</specification-version>
                    </run-dependency>
                </dependency>

Note test locally I've updated the specification version from 1.3.0 to 1.81 since some files specified 1.3.0 and others 1.8.1. Anyway I have no clue why it can't load the module. The actual IGV binary works fine but getting the unit tests to work has always seemed fairly fragile. Any suggestions?

@tkrodriguez
Copy link
Member

I put the initial changes from the update to 20 in #8359 if you want to have a look. In includes an equivalent change to the update center for 20 as well.

@JaroslavTulach
Copy link
Contributor Author

going to update to netbeans 20 at the same time

Integrating my change first, would be "safer".

trouble loading net.java.html

The module should be at netbeans/platform/modules/net-java-html.jar location. If it is there, then I am not sure what's wrong. I'd have to try it. CCing @sdedic

@tkrodriguez
Copy link
Member

I can see it's definitely there and the full IGV works just fine but it's not being loaded in the context of the unit tests. And there's no output explaining why. Presumably there's some step that collects the required modules for running the test and it's being left out but I can't see why. I've tried sprinkling that around in the involved modules but without luck. Is there some trick to getting the dependencies right for the unit tests? It seems like a very manual process.

@JaroslavTulach
Copy link
Contributor Author

JaroslavTulach commented Feb 27, 2024

... initial changes from the update to 20 in #8359 ...

Tom, I am sorry to hear you have problems to upgrade to NetBeans 20. However upgrading to NetBeans 20 doesn't have anything to do with my fix proposed in this PR. Can you help me integrate fix in this PR first and only then seek help of Oracle NetBeans experts like @sdedic, @dbalek, etc. to resolve the upgrading?

I'd prefer to move forward by smaller steps rather than getting stuck while trying to make a huge leap. It is important that IGV keeps up with latest NetBeans releases, but all I need right now is just this PR small bugfix.

Thank you for helping and unblocking an outsider like me.

@tkrodriguez
Copy link
Member

Sorry I was out on vacation last week, but I've come to the same conclusion that I should just update against the current release for now and work out my netbeans issues separately. It is disappointing that the IDE can't tell me what wrong with the test dependencies. Whatever the issues is, it appeared with Netbeans 18. I'll adjust my PR to push your fix along with a few minor fixes.

@JaroslavTulach
Copy link
Contributor Author

JaroslavTulach commented Mar 1, 2024

update against the current release for now

Good!

It is disappointing that the IDE can't tell me what wrong with the test dependencies.
Whatever the issues is, it appeared with Netbeans 18

Yes, this is not really widely used functionality. I assume most of the NetBeans Platform application based projects have already migrated to Maven and regressions in this Ant based support just pass thru the release process unnoticed.

As a NetBeans founder I feel I should be glad to help you, but refreshing/rebooting memory isn't easy and nobody is going to reward me for such mental reboot. OracleLabs employs enough experienced ex-NetBeans engineers. Ask them for help.

@graalvmbot graalvmbot merged commit 3bf0672 into oracle:master Mar 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants