Merge framework master #3
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Build the aarch64 launcher too. Change-Id: I0c1795c56cfdac49797c24a305f3e8a60a14b324 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Add to equinox executable feature. Change-Id: Idf0307f86a98edf7aa562822506e8f1148e37b29 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Change-Id: I0e8e2a2084056757e4b17e5f9bd5eaaeb2c2e89b Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
Change-Id: I9553166260b836f09f7b53b7bd4bcdbd09574810 Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
Change-Id: Ie59ae52508654e24673de0031fde35e7dac44531 Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
constructors Change-Id: I91933a37b055da0b4993a2c52aeedc5054e86541 Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
Change-Id: I17a98703f06162251d60d573832ff23831d55721 Signed-off-by: Sravan Kumar Lakkimsetti <sravankumarl@in.ibm.com>
Change-Id: I4c306e3a41943069c0d0976068f1fe5fef03d696 Signed-off-by: Sravan Kumar Lakkimsetti <sravankumarl@in.ibm.com>
Change-Id: Ia562d850c2b9f3e2e1d8a21bdadff7a73424bea3 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Change-Id: I7d0c50a092ba5a9ef0d626a9ff1093c22408a337 Signed-off-by: Sravan Kumar Lakkimsetti <sravankumarl@in.ibm.com>
Change-Id: I38e565385bfd177df6ebb4248fcf2f9604d51cf8 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Change-Id: Iac952ddf11b9e14c7ce7fcf5a4937bb21ba95c7f Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
There are scenarios where creating a domain will fail, for example if the bundle file has been deleted. There is no need to create a domain in the case of invalidating a wiring because we are just trying to clear the permission cache of the existing domain when invalidating a wiring. Change-Id: Icb5d2118a267d08192f3ead21991d1955d36d34f Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
Change-Id: I9b88c63a78c6c13c712656947cc603ccf55b5196
Change-Id: Ib9b88409b2169523bcf3cfcd72216f114e52616b
Change-Id: If3aa5655cbeb8c2393c42d1789d9464d737b8e9d Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Change-Id: I5a24500bfc38390c26a6e7ca74dc475870ddbbad
Change-Id: I93a88e9570dd04374e0359bbfb22bb3a343aac4b Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
Change-Id: I93ae71de907bef77c3681454590b0652047b1214 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
No need to suppress unchecked when one can use emptyList instead of EMPTY_LIST. Change-Id: Ibdf803adc6f2b2a1d8956862915a4d14e28960aa Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
org.ietf.jgss Change-Id: I88863141b5ecfef8a8ef44631722cede34e44001 Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
Change the typo to test both INCLUDE and EXCLUDE Change-Id: I29fd936ed748144ad2377ba38c2ecc4e7adafaba Signed-off-by: Carsten Hammer <carsten.hammer@t-online.de>
Change-Id: Ibba82884426e7e35df0c49d301ab1cea016a5f56 Signed-off-by: Carsten Hammer <carsten.hammer@t-online.de>
Change-Id: Iaedc19d937a069e40cef242d40b4ac0b746848c0 Signed-off-by: Thomas Watson <tjwatson@us.ibm.com>
…quinox#42 (eclipse-equinox#43) Signed-off-by: Hannes Wellmann <wellmann.hannes1@gmx.net>
It is common for a service lookup to be done with no supplied class name. This is common for ServiceTracker when a Filter object is used during construction. It is also how the Felix SCR implementation looksup services when determining if the requirements are met for a service component. This fix uses the FilterImpl method getRequiredObjectClass to try and determine the class name a service filter is trying to lookup so it can drastically reduce the number of service references that need to be evaluated against the supplied filter.
With this change we assume and FilterImpl that returns a non-null getRequiredObjectClass and has no children is a simple filter only matching on objectClass, for example: (objectClass=SomeService) For this case we can avoid ever calling match on the filter as long as we can lookup services by the required objectClass
Co-authored-by: BJ Hargrave <hargrave@us.ibm.com>
…clipse-equinox#41) And update to latest version of - org.osgi.util.function - org.osgi.util.promise Discussed in issue: https://github.com/eclipse-equinox/equinox.framework/issues/40
This change adjusts the error dialog shown at Eclipse start-up, to be able to display large amount of text. This ensures the dialog shows all information on JVM start-up errors, since then the JVM arguments are listed as a very long info. If the displayed message is short, the old dialog is used. Change-Id: I577b701d2c4bc54a54a39f3cb5b721a4279beb99 Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
In CloseableBundleFile.open(boolean) the caller may request
to keep the lockOpen lock once the bundle file is successfully
opened. Within the try block while the lockOpen is being held
during the open(boolean) call there are cases where it is
possible that the closed flag will be false upon exit if something
unexpected happens (OMM, NPE, etc).
Any time open(boolean) throws any type of throwable we need
to ensure the lockOpen lock is released. If not then methods
like this risk exiting without ever releasing the lock:
public BundleEntry getEntry(String path) {
if (!lockOpen()) {
return null;
}
try {
return findEntry(path);
} finally {
releaseOpen();
}
}
Instead of handling this in open(boolean) the lock/unlock
responsibilities are moved to lockOpen() and open() methods
so we can handle the exceptional case on lockOpen() better.
Now there is an internalOpen method that must be called
while holding the openLock lock.
Previous behavior would throw any other exceptions besides IOExceptoin to the caller. This change brings that behavior back. The unexpected exception is still fired as a framework error event before re-throwing
… for listResources When the OSGi API BundleWiring was added a new way to search for resources was added with the listResources method. The buddy loading policy was not considered with the Equinox implementation of this new listResources method. This address that for only the registered and dependent policies. Other policies involve searching the system class loaders, which we cannot easily list all resources, or the global policy, which searches all exported packages and would lead to a very expensive listing of all bundles. This implementation will only search buddies when the LISTRESOURCES_LOCAL is not used.
…se-equinox#54) * Avoid use of SecureRandom for creating temporary files File.createTempFile uses SecureRandom under the covers to generate unique file names. This may be important for when storing files in the global temporary file space to avoid something guessing the file and writing to it. The usage in the framework for createTempFile is always used for staging file content before it gets (reliably) moved to its final destination. This always happens as an implementation detail of storing some state in the internal file storage of the framework. Risk is low that some malicious code would be able to predict the temporary file and somehow interfere with the behavior of the framework. Once some malicious code has had access to the internal storage of the framework they will be able to write to other files that have very predictable names to cause issues. * Limit retry attempts to MAX_TEMP_NUM (100000) If we really have this many temporary files there is something wrong that needs to be addressed. Fail with IOException if we detect a 100000 attempts to find a non-existing file
* Update Info.plist * Update Info.plist * Update Info.plist
Not needed at all and to be removed via eclipse-pde/eclipse.pde#147
Signed-off-by: Christoph Läubrich <laeubi@laeubi-soft.de>
|
@tjwatson would you please enable merge commits (only PL can do it) ? |
laeubi
force-pushed
the
merge-framework-master
branch
from
4db8281
to
e13ba7a
Compare
Should be enabled now. |
|
The merge LGTM. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.