This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 229823 - Upgrade JNA to version 4.0.0
Summary: Upgrade JNA to version 4.0.0
Status: RESOLVED FIXED
Alias: None
Product: platform
Classification: Unclassified
Component: -- Other -- (show other bugs)
Version: 7.4
Hardware: All All
: P2 normal (vote)
Assignee: Stanislav Aubrecht
URL:
Keywords:
Depends on:
Blocks: 235536
  Show dependency tree
 
Reported: 2013-05-16 08:16 UTC by Stanislav Aubrecht
Modified: 2014-05-15 15:43 UTC (History)
7 users (show)

See Also:
Issue Type: TASK
Exception Reporter:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stanislav Aubrecht 2013-05-16 08:16:44 UTC 
I'm planning to upgrade the JNA library to version 3.5.2 in NetBeans 7.4
The current version is 3.2.7

If there's any reason NOT to upgrade, please state it here.

The change log is available here: https://github.com/twall/jna/blob/master/CHANGES.md
Comment 1 Antonin Nebuzelsky 2013-05-16 15:11:32 UTC 
CC-ed QE and other teams. There is a bunch of areas depending on JNA and after the upgrade, they should all be checked.

E.g.
- window system effects (floating window transparency, dragged window shape)
- file systems - native listening for file changes
- keyring impl
- embedded browser
- subversion support (svnkit, but fallbacks to JNI if needed)
- python shell
- external execution
- kill a tree of processes
- native process execution
- jruby
- CND's usecases

Marian, and others, are you OK with the JNA version upgrade at this stage of 7.4?
Comment 2 Marian Mirilovic 2013-05-16 15:17:36 UTC 
we are fine with that, thanks for notification
Comment 3 Stanislav Aubrecht 2013-05-17 08:34:29 UTC 
It seems that are some incompatible changes in the latest version of JNA. I'm getting this exception on MS Windows:

java.lang.AbstractMethodError: com.sun.jna.Structure.getFieldOrder()Ljava/util/List;
	at com.sun.jna.Structure.fieldOrder(Structure.java:853)
	at com.sun.jna.Structure.getFields(Structure.java:879)
	at com.sun.jna.Structure.deriveLayout(Structure.java:1026)
	at com.sun.jna.Structure.calculateSize(Structure.java:951)
	at com.sun.jna.Structure.calculateSize(Structure.java:918)
	at com.sun.jna.Structure.allocateMemory(Structure.java:357)
	at com.sun.jna.Structure.<init>(Structure.java:199)
	at com.sun.jna.Structure.<init>(Structure.java:188)
	at com.sun.jna.Structure.<init>(Structure.java:175)
	at com.sun.jna.Structure.<init>(Structure.java:167)
	at org.netbeans.modules.masterfs.watcher.windows.WindowsNotifier$FILE_NOTIFY_INFORMATION.<init>(WindowsNotifier.java:164)
	at org.netbeans.modules.masterfs.watcher.windows.WindowsNotifier$FileInfo.<init>(WindowsNotifier.java:283)
	at org.netbeans.modules.masterfs.watcher.windows.WindowsNotifier.addWatch(WindowsNotifier.java:340)
	at org.netbeans.modules.masterfs.watcher.windows.WindowsNotifier.addWatch(WindowsNotifier.java:74)
	at org.netbeans.modules.masterfs.providers.Notifier$1.addWatch(Notifier.java:111)
	at org.netbeans.modules.masterfs.watcher.Watcher$Ext.register(Watcher.java:243)
	at org.netbeans.modules.masterfs.watcher.Watcher.register(Watcher.java:118)
	at org.netbeans.modules.masterfs.filebasedfs.fileobjects.BaseFileObj.addFileChangeListener(BaseFileObj.java:528)
	at org.openide.filesystems.JarFileSystem.setJarFile(JarFileSystem.java:390)
	at org.openide.filesystems.JarFileSystem.<init>(JarFileSystem.java:197)
	at org.netbeans.core.startup.layers.ArchiveURLMapper.getFileSystem(ArchiveURLMapper.java:168)
	at org.netbeans.core.startup.layers.ArchiveURLMapper.getFileObjects(ArchiveURLMapper.java:140)
	at org.openide.filesystems.URLMapper.findFileObject(URLMapper.java:213)
	at org.openide.filesystems.FileUtil.getArchiveRoot(FileUtil.java:1770)
	at org.netbeans.modules.javascript2.editor.classpath.ClassPathProviderImpl.getJsStubs(ClassPathProviderImpl.java:128)
	at org.netbeans.modules.javascript2.editor.classpath.ClassPathProviderImpl.getBootClassPath(ClassPathProviderImpl.java:102)
	at org.netbeans.modules.javascript2.editor.classpath.ClassPathProviderImpl$2.run(ClassPathProviderImpl.java:162)
Comment 4 Stanislav Aubrecht 2013-05-17 12:32:51 UTC 
Because of the incompatible changes we'll postpone the upgrade after the 7.4 release.
Comment 5 Stanislav Aubrecht 2013-09-04 08:44:34 UTC 
core-main de67e30055cd
Comment 6 Quality Engineering 2013-09-05 01:30:24 UTC 
Integrated into 'main-silver', will be available in build *201309050001* on http://bits.netbeans.org/dev/nightly/ (upload may still be in progress)

Changeset: http://hg.netbeans.org/main-silver/rev/de67e30055cd
User: S. Aubrecht <saubrecht@netbeans.org>
Log: #229823 - upgrade JNA lib to 4.0.0
Comment 7 Stanislav Aubrecht 2013-09-06 11:35:03 UTC 
rolled back in core-main e2cfd3469c03
Comment 8 luiz_souza 2013-11-25 19:59:20 UTC 
(In reply to Stanislav Aubrecht from comment #7)
> rolled back in core-main e2cfd3469c03

Sorry to bother, but is this change already on the latest builds? I'm in need of the JNA with this version.

Thank you.
Comment 9 Stanislav Aubrecht 2013-11-26 16:15:05 UTC 
(In reply to luiz_souza from comment #8)
> (In reply to Stanislav Aubrecht from comment #7)
> > rolled back in core-main e2cfd3469c03
> 
> Sorry to bother, but is this change already on the latest builds? I'm in
> need of the JNA with this version.
> 
> Thank you.

It has been postponed due to licensing issues so it's not going to be included in NetBeans 8.0

But if you just need new JNA binaries you can replace them manually - if you keep the naming schema the same.
Comment 10 NeilCSmith 2013-12-12 11:19:09 UTC 
(In reply to Stanislav Aubrecht from comment #9)
> It has been postponed due to licensing issues so it's not going to be
> included in NetBeans 8.0

Could you clarify "licensing issues"? If it's the issue I'm aware of then *not* doing upgrade might be more of an issue?

https://github.com/twall/jna/issues/252
Comment 11 Stanislav Aubrecht 2014-05-15 15:43:18 UTC 
core-main 2ea05bd7083e