Skip to main content
aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-05-15Bug 444188 - Fix concurrent access to listener collectionsS4_8_0_RC4aS4_8_0_RC4S4_8_0_RC3S4_8_0_RC2S4_8_0_RC1R4_8I20180611-0500I20180607-1630I20180607-0300I20180606-2000I20180606-0800I20180605-2000I20180605-0800I20180605-0430I20180604-2000I20180604-0800I20180603-2000I20180603-0800I20180602-1500I20180602-0800I20180601-2000I20180531-0700I20180530-2000I20180530-0800I20180530-0200I20180530-0035I20180529-2345I20180529-2000I20180529-0800I20180529-0600I20180529-0210I20180528-2020I20180528-2000I20180528-0800I20180527-2250I20180527-2000I20180527-0800I20180526-1500I20180526-0800I20180525-2000I20180524-0900I20180523-2000I20180523-0800I20180522-2000I20180522-0800I20180521-2000I20180521-0800I20180520-2000I20180520-0800I20180519-1500I20180519-0800I20180518-2000I20180516-2000I20180516-0800I20180515-2000I20180515-1755I20180515-1620I20180515-1440I20180515-1300I20180515-0950R4_8_maintenanceSravan Kumar Lakkimsetti1-1/+2
Change-Id: Ie68129e470fcfb6b7ab4ada55e456f268d0a9c27
2018-05-12Bug 534597 - Unanticipated comparator errors in I20180511-2000I20180515-0800I20180514-2000I20180514-0800I20180513-2000I20180513-0800I20180512-1500I20180512-0830Sravan Kumar Lakkimsetti8-0/+8
Change-Id: I81a22fab41385acd21a618ae41031e372a9387a6 Signed-off-by: Sravan Kumar Lakkimsetti <sravankumarl@in.ibm.com>
2018-05-02More try-with-resources in simpleconfigurator.S4_8_0_M7I20180511-2000I20180509-2000I20180509-0800I20180508-2000I20180508-0800I20180507-2205I20180507-2000I20180507-0810I20180507-0800I20180506-2000I20180506-0800I20180505-1500I20180504-2000I20180503-2000I20180503-0740I20180502-2000Alexander Kurtakov2-31/+11
Change-Id: I1e45a9d05e0c31b4b0e727310d2af56cef670bdf Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-05-02Make TarFile implement closeable.Alexander Kurtakov2-7/+5
And deal with it using try-with-resources. Change-Id: I0389ab78f2aefe157378f2378f39564deee0c125 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-05-02Try-with-resources and foreach in ConfigApplier.Alexander Kurtakov1-60/+43
Change-Id: I07351432b084b58a9dd1ef4ff907779d2c743a08 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-05-02Translate comment from French to EnglishMickael Istria1-1/+1
Change-Id: I21e7779bcfb7118ca6be1d48a8b7d16ed8aa7752 Signed-off-by: Mickael Istria <mistria@redhat.com>
2018-05-02Bug 534261 - Removed PackageAdmin from the p2 director applicationTodor Boev1-30/+5
Change-Id: Ic11776d49525dfc7968f09cae79ceb5179178046 Signed-off-by: Todor Boev <rinsvind@gmail.com>
2018-04-30Foreach and efficient Map traversal changes.I20180502-0100I20180501-2000I20180430-2000Alexander Kurtakov9-96/+232
Change-Id: I27e76c54d54a7c9d22aec5a124db6f11b7bc6444 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-30Define parent pom in p2 parent pom.Alexander Kurtakov1-0/+7
Ensures all plugins are as defined in pluginManagement. Change-Id: Ie72abd640137d23e4ccf3d813247635ef8a4351a Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-25Update test reconciler product.I20180430-0715I20180429-2000I20180428-2020I20180428-1500I20180427-2000I20180426-2000I20180426-0500I20180425-2000I20180425-1705I20180425-1650I20180425-1455Alexander Kurtakov1-3/+14
Let it run the reconciler app instead of workbench. Add all required plugins. Change-Id: Icc15f4953f821592d596c4bd10f748d36c88cb3a Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-25Add bundle vendor to o.e.equinox.frameworkadmin.test bundle.Alexander Kurtakov3-2/+17
And externalize the strings. Change-Id: I746bf1f6f437f517007bfc8b60af1bbad9e94940 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-25Remove unused method.Alexander Kurtakov1-27/+2
That causes import of deprecated PackageAdmin. Change-Id: Idb36c049f672d9b8cb17f1fe818c73372133fc4b Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-25Cleanups.I20180425-0910I20180425-0725Alexander Kurtakov8-35/+31
* Foreach conversion. * Removed useless non-javadoc comments. * Use isEmpty. Change-Id: I86a3291f3abf30a8dee2b4a4c008978b24ec75d3 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-25Touching reconciler product after move to Tycho 1.2.0I20180425-0505Sravan Kumar Lakkimsetti1-0/+1
Change-Id: I7ac3fbc380298c2462abe4c84702a16d2bfe9c54 Signed-off-by: Sravan Kumar Lakkimsetti <sravankumarl@in.ibm.com>
2018-04-18Bug 533758 - Direct use of IStructuredSelectionI20180425-0250I20180424-2000I20180423-2000I20180423-0655I20180423-0640I20180422-2000I20180421-1500I20180420-2000I20180419-2000I20180418-2000Alexander Kurtakov13-31/+46
Instead of getSelection and cast. Enable save actions to auto remove useless casts. Change-Id: Ie9fd9546ac74d146a9fa219e37d012d7a3e79415 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-16Bug 533617 - NLS missing message warnings since 4.8 M6I20180417-2000I20180416-2000Alexander Kurtakov2-0/+5
Restore MD5 error messages. They are still used in deprecated code but were errorneously removed. Change-Id: Ibf3c499075b710f640032605cf3f8d9d90b3f6bf Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-13Bug 533117 - setRepositoryProperty: fixed regression when value is nullI20180416-0305I20180415-2000I20180414-1500I20180413-2000Karsten Thoms1-3/+8
A null value should remove the entry from the repositoryProperties. But when repositoryProperties is not initialized a null value was actually added for the key. Changed conditions to first prove wether value is null and only remove the property, when field repositoryProperties is initialized. Set field to null when the last entry was removed. Change-Id: I766094f09d24bcf3c0da093202724ab5c2f0e417 Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-04-09Bug 527874 - Remember failed repositories on load requestI20180412-2000Karsten Thoms1-20/+41
When a loading a repository failed once, its location is remembered to early exit on subsequent load requests. Multiple requests can happen e.g. during Check for Updates action. Change-Id: I56258d50523d95dff84ef4153bfc687f3bf80355 Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-03-27Bug 532923 - Copyright property page does not resizeI20180411-2000I20180411-0735I20180411-0530I20180410-2000I20180409-2000I20180408-2000I20180407-1500I20180406-2000I20180405-2000I20180404-2000I20180403-2000I20180402-2000I20180401-2000I20180331-1500I20180330-2235I20180330-2000I20180329-2000I20180329-0000I20180328-2000I20180327-2000Michael Keppler1-1/+1
Let the main label control grab the horizontal excess space to resize with the property page. The same flag was already set for the nearly identical license property page. Change-Id: I7067f24bec245bc28a825630ad591b35b09306fa Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug:532923
2018-03-20Bug 531345 - Publish product osgi.ee requirement according to <vm>I20180327-0805I20180327-0630I20180326-2000I20180325-2000I20180324-1500I20180323-2000I20180323-0355I20180322-2000I20180322-0645I20180322-0225I20180321-2000I20180320-2000Mickael Istria5-48/+142
Use the EE defined in the <vm> node of product definition to define some osgi.ee capability requirements in product metadata. Add tests to make sure JRE is still included while unit isn't directly referenced by product requirements. Change-Id: Ic2bd5c30215e9936201c70f7268e5c6068979405 Signed-off-by: Mickael Istria <mistria@redhat.com>
2018-03-20Bug 532652 - MetadataFactory.createRequirement ignores descriptionEd Merks1-1/+1
Change-Id: Ie6f6c2ca6abad2d60c1cdbc18c085d60bf7b704d Signed-off-by: Ed Merks <ed.merks@gmail.com>
2018-03-13Bug 532400 - Remove buckminster relengI20180319-2000I20180318-2000I20180317-1500I20180316-2000I20180316-0740I20180316-0730I20180316-0525I20180316-0510I20180316-0455I20180315-2000I20180314-2000I20180313-2000Alexander Kurtakov4-135/+0
Not used for ages. Change-Id: I1b849d554ed5c1fbb3ace4267876fee85294c7ea Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-13Bug 512323 - Missing newer target files in org.eclipse.equinox.p2.relengAlexander Kurtakov13-345/+0
Now that we have photon target file nuke all the old and useless stuff - ancient target files and psf files pointing to the years dead cvs repos. Change-Id: Iec77b4cfeb218be0321e4e8f5937134262a27f38 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-13Bug 512323 - Add target file for building against Photon (4.8).Roland Grunberg1-0/+19
Change-Id: If90e85f1a54e14246bb2d60aa4674b9d8543bf7b Signed-off-by: Roland Grunberg <rgrunber@redhat.com>
2018-03-12Bug 532118 - Eliminate exception when checking if buffer is full in ↵Mykola Nikishov1-6/+4
MessageDigestProcessingStep There is common wisdom behind the claim that throwing exception may hurt performance but I have no real numbers to say it will have real impact in this case. However, in this case it is easy to avoid exception by explicitly checking if the buffer is full before putting new data into it. Change-Id: Ifbd178aff4f943c31772aa357ee8369efcee558f Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-03-12Bug 532118 - Reduce number of calls to java.security.MessageDigest.update(byte)Mykola Nikishov3-36/+80
MD5Verifier and ChecksumVerifier are responsible for calculating artifact's checksum and must extend ProcessingStep to fulfill API requirements. ProcessingStep is effectively an unbuffered OutputStream with a method to process a single byte of artifact's data. In respect to MessageDigest, this means updating digest with every new single byte of data. So, for 16 Kb of data, both classes will call MessageDigest's update(byte) 16K times while only 1 call is enough. Introduce MessageDigestProcessingStep that accumulates incoming bytes in the internal buffer and immediately forwards them to the destination stream. As soon as buffer is full, it will update MessageDigest. AbstractBufferingStep could be used as a base class instead but it is too risky to adapt it. java.nio.ByteBuffer fits the bill almost ideally. One caveat is it communicates 'buffer is full' by throwing BufferOverflowException. There are no specific reasons for buffer's size of 16K except an assumption that it should not cause any problems: installing 1000 artifacts with 2 checksums per artifact would increase memory footprint for about 3200K. Similar approach has been used already [1], [2], claiming 5x speedup. This should make [3] less of a problem. [1] 5b2f061cd8ecf96a37783657e7ee7a0110c9d26d [2] Bug 405716 [3] Bug 532036 Change-Id: I1ba0d36267b3f07103c22b93f92c41765e33c8cc Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-03-09Bug 519085 Reduced memory consumption of SimpleArtifactDescriptorI20180312-2000I20180311-2000I20180310-1500I20180309-2000Karsten Thoms2-8/+34
Also of OrderedProperties by interning keys. Change-Id: Ic86797dd633b0537e99d86aad89de26f1c80e497 Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-03-09Bug 448789 - Improved error logging for remediationOperation null refKarsten Thoms1-2/+8
Avoids the NPE as reported in the bug and logs the situation. Change-Id: I4ea13466ec7c764b6d75c9b28c0cba57fc75c301 Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-03-08Bug 532190 - Buffered writingKarsten Thoms1-1/+1
Change-Id: I4630b8206e2f404573ea0fe971eb21e1aae49832 Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-03-07Bug 532094 - Update releng with new hamcrest and apache batikS4_8_0_M6I20180308-0630Alexander Kurtakov1-0/+2
Change-Id: I30f7e43699e28662f8740b41b2318247ea19d98b Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-05Bug 531606 - RequiredCapability introduced binary-incompatible changesI20180307-2000I20180307-1025I20180307-0900I20180307-0825I20180307-0800I20180306-2000I20180306-0800I20180305-2000Mykola Nikishov1-0/+2
p2 tries to not change internals (due to bad downstreams using internals widely) but these are internals after all and considering the churn with capabilities work and current stable state, we are not going to do any work here. Clients would have to adapt to the changes when moving to Photon p2. To communicate this, explicitly mark RequiredCapability as not intended to be referenced by clients. Change-Id: I4bddb36f13178ec07752ee92b6bacaca26370bf2 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-03-05Bug 423715 - Really preserve legacy MD5 propertiesI20180305-0800I20180305-0300Mykola Nikishov4-41/+34
Workflow to calculate checksums and store them in artifact's properties assumes usage of two methods from org.eclipse.equinox.internal.p2.artifact.processors.checksum.ChecksumUtilities: - calculateChecksums(File, Map<String, String>, Collection<String>) - checksumsToProperties(String, Map<String, String>) Two users of these methods, PublisherHelper and RecreateRepositoryApplication handled legacy MD5 properties in an inconsistent way. Even more, RecreateRepositoryApplication effectively ignored legacy MD5 property when recreating artifact descriptor. Move logic to handle legacy MD5 properties from PublisherHelper and RecreateRepositoryApplication to ChecksumUtilities' checksumsToProperties(String, Map<String, String>). p2 codebase has no tests for RecreateRepositoryApplication but PDE Build does, specifically P2Tests' testBug265564. This time PDE Build detected such regression [1], but it would be better to have some test harness for RecreateRepositoryApplication in p2 itself and not rely on downstream projects. [1] Bug 531931 Change-Id: Ic651df754691c25ee824bb444eff251f64f0757a Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-03-02Bug 423715 - Fix since tagsI20180304-2000I20180304-0800I20180303-1500I20180303-1200I20180303-0800I20180303-0335I20180303-0155I20180303-0055I20180302-2000Alexander Kurtakov1-2/+2
Change-Id: I8a5532c9ef2a12cc13e125b1545b466472ec4e0c Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-02Bug 531917 - Remove redundant type parametersAlexander Kurtakov60-210/+387
With Java 8 and properly generified some apis it's not needed to specify the types anymore. Change-Id: Ib758c6f50b1b1fd3f56bd52eb7e664b89b6cc776 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-02Bug 423715 - Fix compile warnings Alexander Kurtakov1-0/+1
In the nightly build there is: 1. WARNING in /src/org/eclipse/equinox/internal/p2/artifact/processors/checksum/ChecksumUtilities.java (at line 139) MD5Verifier checksumVerifier = new MD5Verifier(md5); Resource leak: 'checksumVerifier' is never closed It's stored and used so shouldn't be closed at this point. Change-Id: I9cba78e8706c3122efc5ae1bc1c9c88c1eeec905 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-03-01Bug 423715 - Support multiple algorithms for artifact checksumsI20180301-2000Mykola Nikishov32-92/+1018
Developer of p2-based software should be able to check integrity of artifact checksums using any MessageDigest implementation available, at his own discretion, without ever touching p2 internals. For the purpose of this bug, limit enabled checksum algorithms to: - MD5 (legacy one, deprecated now, for backward compatibility only) - SHA-256 (more collision-resistant than MD5) See artifactChecksums.exsd for more details. First, encode ID of the checksum algorithm into the name of the property itself so that artifact descriptor now may look like: <artifact classifier='binary' id='testKeyId' version='1.2.3'> <properties size='6'> <property name='download.md5' value='b3788632488d48b850255acf68669651'/> <property name='artifact.md5' value='b3788632488d48b850255acf68669651'/> <property name='download.checksum.sha-256' value='d594816b995d1689c2dfc97dc244859abe6bdb9ebeb4b396e401afd85a97ee16'/> <property name='download.checksum.whirlpool' value='e4f3ece3d3f289cc2686a68f0b1b5a2d03da3a8ccdc3cd6d03209e4c789af724c8fa915bb890079e1abe78df44875cec132885dd6ae1176eed7938dfb3c7b551'/> <property name='artifact.checksum.sha-256' value='d594816b995d1689c2dfc97dc244859abe6bdb9ebeb4b396e401afd85a97ee16'/> <property name='artifact.checksum.tiger' value='462a72a1a593b9e2de8721b8d79335fd03e974f2e2e1f35a'/> </properties> </artifact> Prefix helps to avoid conflicts with other, not checksum-related, properties. To find checksums of specific type, we need all properties with prefix 'artifact.checksum.', Extracting checksum id from the name of the property is also trivial. Store these prefixes in IArtifactDescriptor's DOWNLOAD_CHECKSUM and ARTIFACT_CHECKSUM. ChecksumUtilities provides internal API to deal with properties. Second, map ID of checksum algorithm to the name of MessageDigest implementation with a contribution to a new extension point org.eclipse.equinox.p2.artifact.repository.artifactChecksums: <extension point="org.eclipse.equinox.p2.artifact.repository.artifactChecksums"> <artifactChecksum algorithm="SHA-256" id="sha-256" ... /> </extension> Number of actual checksums in artifact descriptor depends on the configuration of the application that created such descriptor because p2 will: - handle MD5 checksums as usual, preserving backward compatibility - calculate other checksums using extensions that contribute to artifactChecksums extension point Last, we use id of specific checksum to get MessageDigest instance with getInstance(String). There is a number of standalone applications (like MirrorApplication) and Ant tasks (like ValidateTask) that allow user to chose specific comparator by id and ArtifactComparatorFactory is responsible for instantiating it. Legacy MD5 comparator requires no special configuration while the new ArtifactChecksumComparator is more generic and needs some parameters to instantiate. ArtifactComparatorFactory handles this by accepting fake comparator id - concatenated ArtifactChecksumComparator's id and checksum algorithm id. In other words, to compare artifacts using SHA-256 algorithm use 'org.eclipse.equinox.artifact.comparator.checksum.sha-256' as comparator id. We generate, consume and compare artifact checksums using all enabled algorithms. Ideally, configuration options should provide more control for both repository publisher and p2 client like: - a priority (compare with SHA-512 first, then SHA-256 and, finally, MD5) - skip specific checksum (do not use MD5 even if provided) - mandatory checksum (require Whirlpool and it's an error if its not available) While I have no good solution for this right now, there are some configuration options provided. Also, number of new enabled checksum algorithms is limited to SHA-256 only (more checksums = more reasons to fail = more reasons to disable). To turn checksum verification off completely: - for p2 client, use property 'eclipse.p2.checksums.disable' (see SimpleArtifactRepository.CHECKSUMS_ENABLED constant). - for p2 publisher, the old PublisherInfo's setArtifactOptions(int) and IPublisherInfo.A_NO_MD5 still could be used. To disable MD5 checksums only, existing properties 'eclipse.p2.MD5Check' and 'eclipse.p2.MD5ArtifactCheck' still can be used. Change-Id: Iacd267e13d5d096694001d34cafbaea5451e7157 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-26Bug 218650 - Parse the product <vm> settingI20180228-2000I20180227-2000Mickael Istria4-1/+105
Change-Id: I2aa1af9333619e0a078c8b8a1e7c90960876197d Signed-off-by: Mickael Istria <mistria@redhat.com>
2018-02-22Bug 531430 - Replace usage of SubProgressMonitor by SubMonitorI20180226-2000I20180225-2000I20180224-1500I20180223-2000I20180222-2000Karsten Thoms8-10/+10
Change-Id: Iea476f0ceb12d83224e0274c1d6a53c49b81adce Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-02-21Bug 531354 - Restore binary compatibility of publisher.eclipse bundleI20180221-2000Alexander Kurtakov1-15/+6
Fix compile warnings. Change-Id: Ia75aff8e0f2f76f6974cf35cb86d1ea8c1ceaa3e Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-02-20Bug 531354 - Restore binary compatibility of publisher.eclipse bundleI20180220-2000Mykola Nikishov1-1/+11
org.eclipse.equinox.p2.publisher.eclipse.BundlesAction is a public not final class exported by publisher.eclipse bundle and there are no restrictions like x-friends, @noextend an alike. This makes it a public API. At binary level, changing signatures [1] of BundlesAction's protected methods is a binary-incompatible change: - remove old methods with first parameter of type ArrayList<IRequirement> - add new methods with the same name but with first parameter of type List<IRequirement> Instead of increasing a major segment of bundle's version, restore binary compatibility by restoring original methods. [1] 02e018c88f5307fc754ae8bc0c0467eb209dd982 Change-Id: I0c5cff87d96c58da45f98d848572271f817364e0 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-15Bug 507657 - Refactor AbstractRepositoryManager's loadIndexFile(URI,I20180219-2000I20180218-2000I20180217-1500I20180216-2000I20180216-0415I20180215-2000Mykola Nikishov1-26/+25
String, IProgressMonitor) - extract methods to load local and remote p2.index file - rename method isURL to isInMemoryRepository and drop explanation on the caller's side Change-Id: I9558d1746b4f4433a63040ab8c9cae18da4a6bdd Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-15Bug 516442 - remove unnecessary import package David Williams1-1/+0
the purpose is to avoid Access Restriction error in workbench in org.eclipse.equinox.p2.ui Change-Id: I1f02be8ce58344f38c4b95ae2f639a8e3499d97e Signed-off-by: David Williams <david_williams@acm.org>
2018-02-15Bug 530592 - Fixed typo in temp file nameKarsten Thoms1-2/+2
Change-Id: I86648442f44dfbc1abf1293a7c9dd895cf46c98f Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-02-15Bug 423715 - Replace StringBuffer with StringBuilder in MD5VerifierMykola Nikishov1-1/+1
Change-Id: Ie5a180d179facbdb532aad3d887ca962f78a2105 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-15Bug 423715 - Use enhanced loop in MD5VerifierMykola Nikishov1-3/+3
Change-Id: I5c134ad37433f7868defe060b34fe40a8d8e820a Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-15Bug 423715 - More javadoc for ArtifactComparatorFactoryMykola Nikishov1-1/+6
Change-Id: I72b1dc794d8da8acc240c380df33f938afe406a7 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-14Bug 423715 - ArtifactComparatorFactory reports available comparatorsMykola Nikishov2-3/+6
Method getArtifactComparator(String) should report which artifact comparators are available. When troubleshooting, user of standalone applications and Ant tasks will have more information. Change-Id: Iba7ea48c139c5ae365bc898804b38dc7cd75cc55 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-14Bug 423715 - ArtifactRepositoryValidator reports original causeMykola Nikishov1-1/+1
When throwing ProvisionException as a result of the underlying IllegalArgumentException from ArtifactComparatorFactory's getArtifactComparator(String), do not hide an original problem but report it to the caller. Change-Id: Iefcf90ea183a24eca48caa4b9006493ca4e356ea Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-12Bug 423715 - Extract methods in ArtifactComparatorFactoryI20180214-2000I20180213-2000I20180213-0125I20180212-2000Mykola Nikishov1-23/+34
...to isolate independent parts of code and make it easier to understand. Also, add javadoc to the single public method. Change-Id: Ie61a36eca7b98fb33af2f267ad58ce4724053f17 Signed-off-by: Mykola Nikishov <mn@mn.com.ua>
2018-02-12Bug 423715 - ArtifactRepositoryValidator should catch IAE in order toMykola Nikishov2-4/+5
throw ProvisionException Check for comparatorId == null is useless because ArtifactComparatorFactory's getArtifactComparator never returns null but throws IllegalArgumentException instead. This way the underlying IAE is leaking to caller and ProvisionException (expected by client) is never thrown. Change-Id: Ib590f6259e31713f7f9601061deef34e56a0552d Signed-off-by: Mykola Nikishov <mn@mn.com.ua>

Back to the top