Hi, i didn't test chmod yet (dont need that under windows) but i tested:
instructions.install = \ org.eclipse.equinox.p2.touchpoint.eclipse.setJvm(jvm:${artifact.location}\\jre\\bin\\j9vm\\jvm.dll); and that doesn't work because the ParameterizedProvisioningAction.processVariables(String, Map<String,Object>, boolean) line: 106 doesn't have that replacement at all the parameter list that is has are: agent=org.eclipse.equinox.internal.p2.core.ProvisioningAgent@107476e3 forced=false profile=Profile(SDKProfile) phaseId=install iu=[R]com.servoy.jre.win32.x86_64.feature.feature.group 12.0.0 installFolder=C:\temp\eclipse context={artifactRepos=[], metadataRepos=[], properties={org.eclipse.equinox.p2.director.followArtifactRepositoryReferences=true, org.eclipse.equinox.p2.director.followRepositoryReferences=null}} manipulator=org.eclipse.equinox.internal.p2.touchpoint.eclipse.LazyManipulator@f4fdbe75 profileDataDirectory=C:\temp\eclipse\p2\org.eclipse.equinox.p2.engine\profileRegistry\SDKProfile.profile\.data platformConfiguration=org.eclipse.equinox.internal.p2.touchpoint.eclipse.PlatformConfigurationWrapper@734dac8 _p2_internal_last_result_variable_=org.eclipse.equinox.p2.engine.spi.Value@565ca0eb operand=null --> [R]com.servoy.jre.win32.x86_64.feature.feature.group 12.0.0 sourceBundles=org.eclipse.equinox.internal.p2.touchpoint.eclipse.SourceManipulator@d99d49a8 you see installFolder is there, but not "artifact.location" i will have a look how that then supposed to work for chmod By the way are you not hitting this bug? https://bugs.eclipse.org/bugs/show_bug.cgi?id=551378 On Sat, 9 Nov 2019 at 10:43, Torkild U. Resheim <torki...@gmail.com> wrote: > Hi, > > It seems we had a similar issue at some point. The code that is currently > in use and has been for years looks like this: > > instructions.install.import= \ > org.eclipse.equinox.p2.touchpoint.natives.chmod > > instructions.install = \ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/ControlPanel,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/java,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/javaws,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/jcontrol,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/jjs,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/keytool,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/orbd,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/pack200,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/policytool,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/rmid,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/rmiregistry,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/servertool,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/tnameserv,permissions:755);\ > > chmod(targetDir:${artifact.location},targetFile:jre/bin/unpack200,permissions:755); > > I don't know why we are not using the "-R" option. > > Best regards, > Torkild > > > 8. nov. 2019 kl. 22:07 skrev Mickael Istria <mist...@redhat.com>: > > > > > > > > On Fri, Nov 8, 2019 at 5:37 PM Johan Compagner <jcompag...@gmail.com> > wrote: > > I guess it should work for that target dir. But what about the vm > setting? That's just one argument that needs to then be concat to each > other. > > > > But I will have ab test if I can seer how the code works for this > argument.. I expected it to be more like a variable. (Like installDir) > > > > Looking in the code of the Eclipse touchpoints (including setJVM and > chmod), there seems to be a ${artifact.location} property you can use for > this purpose. > > So you could maybe use > > > > > org.eclipse.equinox.p2.touchpoint.eclipse.setJvm(jvm:${artifact.location}/jre/Contents/Home/bin);\ > > > org.eclipse.equinox.p2.touchpoint.eclipse.chmod(targetDir:${artifact.location}/jre/Contents/Home/,targetFile:bin,permissions:755,options:-R); > > _______________________________________________ > > p2-dev mailing list > > p2-dev@eclipse.org > > To change your delivery options, retrieve your password, or unsubscribe > from this list, visit > > https://www.eclipse.org/mailman/listinfo/p2-dev > > _______________________________________________ > p2-dev mailing list > p2-dev@eclipse.org > To change your delivery options, retrieve your password, or unsubscribe > from this list, visit > https://www.eclipse.org/mailman/listinfo/p2-dev >
_______________________________________________ p2-dev mailing list p2-dev@eclipse.org To change your delivery options, retrieve your password, or unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/p2-dev