Merged to master, thanks!
From: David Nyström [mailto:[email protected]] Sent: Tuesday, September 04, 2012 1:23 PM To: [email protected]; Zhang, Jessica Cc: Björn Arnelid Subject: RE: [yocto] [eclipse-poky][PATCH] Fix NullPointerException Hi, The Nullpointer exception is thrown when trying to use a toolchain tarball which has DISTRO set to something else than poky. This would generate the name *${DISTRO}sdk* in the sysroot under /opt/${DISTRO}/1.2/sysroots/ Br, David Sent from my Android phone using TouchDown (www.nitrodesk.com<http://www.nitrodesk.com>) -----Original Message----- From: Zhang, Jessica [[email protected]] Received: Tuesday, 04 Sep 2012, 19:21 To: David Nyström [[email protected]]; [email protected]<mailto:[email protected]> [[email protected]] CC: Björn Arnelid [[email protected]] Subject: RE: [yocto] [eclipse-poky][PATCH] Fix NullPointerException Hi Björn, Thanks for the patch, there's no issue as to the coding and implementation of the patch. Just need to clarify what's the usage scenario or how did you generate your toolchain that's different from the naming convention of the sysroot directory names that's generated by poky build? Thanks, Jessica -----Original Message----- From: [email protected]<mailto:[email protected]> [mailto:[email protected]] On Behalf Of David Nystr?m Sent: Tuesday, September 04, 2012 1:21 AM To: [email protected]<mailto:[email protected]> Cc: Björn Arnelid Subject: [yocto] [eclipse-poky][PATCH] Fix NullPointerException From: Björn Arnelid <[email protected]<mailto:[email protected]>> [Yocto Bugzilla] -- https://bugzilla.yoctoproject.org/show_bug.cgi?id=3019 When using a Toolchain where sysroot is not named <arch>-pokysdk-linux a NullPointerException is thrown when trying to apply changes in Yocto Preferences. It should be possible to use Toolchains with a sysroot named <arch>-<organizationName>sdk-linux with the plugin. static private String findHostArch(File sysroot_dir) now accept folders named *sdk-linux instead of *-pokysdk-linux. checkYoctoSDK will also return SDKCheckResults.TOOLCHAIN_NO_SYSROOT if findHostArch returns null. Author: Björn Arnelid <[email protected]<mailto:[email protected]>> Signed-off-by: David Nyström <[email protected]<mailto:[email protected]>> --- .../src/org/yocto/sdk/ide/YoctoSDKUtils.java | 20 +++++++++++++++----- 1 file changed, 15 insertions(+), 5 deletions(-) diff --git a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/YoctoSDKUtils.java b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/YoctoSDKUtils.java index 6be90be..749fac7 100644 --- a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/YoctoSDKUtils.java +++ b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/YoctoSDKUtils.java @@ -100,7 +100,17 @@ public class YoctoSDKUtils { File sysroot_dir = new File(sysroot_dir_str); if (!sysroot_dir.exists()) return SDKCheckResults.TOOLCHAIN_NO_SYSROOT; - String toolchain_host_arch = findHostArch(sysroot_dir); + + String toolchain_host_arch = null; + + try + { + toolchain_host_arch = findHostArch(sysroot_dir); + } + catch(NullPointerException e) + { + return SDKCheckResults.TOOLCHAIN_NO_SYSROOT; + } if (!toolchain_host_arch.equalsIgnoreCase(platform)) { if (!platform.matches("i\\d86") || !toolchain_host_arch.matches("i\\d86")) @@ -576,7 +586,7 @@ public class YoctoSDKUtils { static private String findHostArch(File sysroot_dir) { FilenameFilter nativeFilter = new FilenameFilter() { public boolean accept(File dir, String name) { - if (name.endsWith("-pokysdk-linux")) { + if (name.endsWith("sdk-linux")) { return true; } else { return false; @@ -590,11 +600,11 @@ public class YoctoSDKUtils { for (File file : files) { if (file.isDirectory()) { String path = file.getName(); - arch = path.substring(0, path.indexOf("-pokysdk-linux")); + String[] subPath = path.split("-"); + arch = subPath[0]; } else continue; } return arch; } -} - +} \ No newline at end of file -- 1.7.9.5 _______________________________________________ yocto mailing list [email protected]<mailto:[email protected]> https://lists.yoctoproject.org/listinfo/yocto
_______________________________________________ yocto mailing list [email protected] https://lists.yoctoproject.org/listinfo/yocto
