No.  Reverting can be messy.  I prefer we just keep moving forward ...   It
is not a crisis if master isn't perfect, as long as we can fix it
reasonably quickly :)


On Fri, Jul 9, 2021 at 3:03 PM David Cromberge <
[email protected]> wrote:

> Interesting, I have not yet tested the enclose compatibly in that manner.
> I tested using the IDE itself.
>
> As an update, I have created a new toolchain for the user, and am looking
> at the surefire plugin mechanism for scanning dependencies.
>
> I’ll add the mvn eclipse:eclipse command to the list - I agree, it should
> not be scanning maven central.
>
> Would you prefer I revert the PR merge whilst I address these issues?
>
>
> David
>
> On 9 Jul 2021, at 22:55, leerho <[email protected]> wrote:
>
> Ok, I removed the <classifier>tests</classifier> from the pom.
>
> As long as the entries in the toolchain match the JDKs I have installed
> and the matrix versions also match then
>
> mvn clean install --toolchains .github/workflows/.toolchains.xml  WORKS!
>
>
> However, when I try: mvn clean eclipse:eclipse --toolchains
> .github/workflows/.toolchains.xml
> it fails with
>
>    - [*ERROR*] Failed to execute goal
>    org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce
>    *(enforce-banned-dependencies)* on project
>    datasketches-memory-java8-tests: *Execution
>    enforce-banned-dependencies of goal
>    org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce failed:
>    org.apache.maven.shared.dependency.graph.DependencyGraphBuilderException:
>    Could not resolve following dependencies:
>    [org.apache.datasketches:datasketches-memory-java8:jar:2.0.0-SNAPSHOT
>    (test)]*: Could not resolve dependencies for project
>    org.apache.datasketches:datasketches-memory-java8-tests:jar:2.0.0-SNAPSHOT:
>    org.apache.datasketches:datasketches-memory-java8:jar:2.0.0-SNAPSHOT was
>    not found in
>    
> https://repository.apache.org/content/groups/snapshots/org/apache/datasketches/
>    
> <https://www.google.com/url?q=https://repository.apache.org/content/groups/snapshots/org/apache/datasketches/&source=gmail-imap&ust=1626472557000000&usg=AOvVaw2ropAlkfYzKFBNVxduUwHm>
>    during a previous attempt. This failure was cached in the local repository
>    and resolution is not reattempted until the update interval of
>    apache.snapshots has elapsed or updates are forced
>    - The irony is that I think
>    *org.apache.datasketches:datasketches-memory-java8:jar:2.0.0-SNAPSHOT 
> *exists
>    in my .m2:
>    (base) datasketches $ pwd
>    /Users/lrhodes/.m2/repository/org/apache/datasketches
>    (base) datasketches $ tree
>    .
>    ├── datasketches-memory
>    ...
>    │   ├── 2.0.0
>    │   │   ├── datasketches-memory-2.0.0.jar.lastUpdated
>    │   │   └── datasketches-memory-2.0.0.pom.lastUpdated
>    │   ├── 2.0.0-SNAPSHOT
>    │   │   ├── _remote.repositories
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT-javadoc.jar
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT-sources.jar
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT-test-sources.jar
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT-tests.jar
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT.jar
>    │   │   ├── datasketches-memory-2.0.0-SNAPSHOT.pom
>    │   │   └── maven-metadata-local.xml
>    │   ├── maven-metadata-apache.releases.https.xml
>    │   ├── maven-metadata-central.xml
>    │   ├── maven-metadata-central.xml.sha1
>    │   ├── maven-metadata-local.xml
>    │   └── resolver-status.properties
>    ├── datasketches-memory-java8
>    │   └── 2.0.0-SNAPSHOT
>    │       ├── datasketches-memory-java8-2.0.0-SNAPSHOT.jar.lastUpdated
>    │       └── resolver-status.properties
>    ├── datasketches-memory-java8-tests
>    │   └── 2.0.0-SNAPSHOT
>    │       ├──
>    datasketches-memory-java8-tests-2.0.0-SNAPSHOT-tests.jar.lastUpdated
>    │       └── resolver-status.properties
>    ...
>
> However when I open the contents of one of those "lastUpdated" files I see:
>
>    - #NOTE: This is a Maven Resolver internal implementation file, its
>    format can be changed without prior notice.
>    #Fri Jul 09 11:14:59 PDT 2021
>    https\://
>    
> repository.apache.org/content/groups/snapshots/org/apache/datasketches/.lastUpdated=1625854499312
>    
> <https://www.google.com/url?q=http://repository.apache.org/content/groups/snapshots/org/apache/datasketches/.lastUpdated%3D1625854499312&source=gmail-imap&ust=1626472557000000&usg=AOvVaw0V0lZQnMhTvqogOHBwW7Xh>
>    https\://
>    
> repository.apache.org/content/groups/snapshots/org/apache/datasketches/.error=
>    
> <https://www.google.com/url?q=http://repository.apache.org/content/groups/snapshots/org/apache/datasketches/.error%3D&source=gmail-imap&ust=1626472557000000&usg=AOvVaw1bOghQQaqPVA0g29hbDYSg>
>    (blank)
>
> So it appears to reference the apache repo! Which it should not, I would
> think.
>
> Even though I can get maven to run, so far I haven't been able to get
> Eclipse to run.
>
> Lee.
>
>
> On Fri, Jul 9, 2021 at 1:18 PM David Cromberge <
> [email protected]> wrote:
>
>> Lee,
>>
>> Apologies for the issues that you are experiencing with the build.
>>
>> In order to support multiple versions of the JVM, the multi-release JAR
>> is tested using the java8 tests suite.  The tests are required as a module
>> dependency and reflection is used to run all the tests from the java8 tests
>> JAR.
>> This process requires the use of a dependency classifier which is
>> responsible for these errors.
>>
>> As a workaround, you can temporarily install the dependency locally,
>> using `mvn install`, or you can manually edit the datasketches-memory-tests
>> pom.xml to remove the classifier:
>>
>> <Screenshot 2021-07-09 at 21.16.53.png>
>>
>>
>> Admittedly, it’s not ideal to use the GitHub actions toolchains as a user
>> template.
>>
>> I’ll look into these problems in the interim and create a follow-up pull
>> request to address them,
>>
>> Thank you for your patience,
>> David
>>
>>
>> On 9 Jul 2021, at 20:54, leerho <[email protected]> wrote:
>>
>> Hmmm.
>> OK, :-(  I read the README, and it says I only need JDK 8,9 and 11
>> installed.
>>
>> With JAVA_HOME pointing to JDK 11, I tried
>>  mvn clean compile --toolchains .github/workflows/.toolchains.xml
>>
>> I get the following errors:
>>
>>    - [*ERROR*] Misconfigured toolchain.
>>    *org.apache.maven.toolchain.MisconfiguredToolchainException*: 
>> *Non-existing
>>    JDK home configuration at
>>    /Users/lrhodes/dev/git/Apache/datasketches-memory2/${env.JAVA10_HOME}*
>>    - [*ERROR*] Misconfigured toolchain.
>>    *org.apache.maven.toolchain.MisconfiguredToolchainException*: 
>> *Non-existing
>>    JDK home configuration at
>>    /Users/lrhodes/dev/git/Apache/datasketches-memory2/${env.JAVA12_HOME}*
>>
>> I think it is telling me I need 10 and 12 installed too.
>> So I temporarily removed 10 and 12 from the matrix test (I don't have
>> those installed),
>> and I removed 10 and 12 from the toolchain.
>>
>> Now it gives me this error:
>>
>>    - [*ERROR*] Failed to execute goal
>>    org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce
>>    *(enforce-banned-dependencies)* on project datasketches-memory-tests:
>>
>> *Execution enforce-banned-dependencies of goal  *
>>
>> *org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce failed:
>>    org.apache.maven.shared.dependency.graph.DependencyGraphBuilderException:
>>    Could not resolve following dependencies:
>>    
>> [org.apache.datasketches:datasketches-memory-java8-tests:jar:tests:2.0.0-SNAPSHOT
>>    (test)]*:
>>    Could not resolve dependencies for project
>>    org.apache.datasketches:datasketches-memory-tests:jar:2.0.0-SNAPSHOT:
>>    
>> org.apache.datasketches:datasketches-memory-java8-tests:jar:tests:2.0.0-SNAPSHOT
>>
>>    was not found in
>>    
>> https://repository.apache.org/content/groups/snapshots/org/apache/datasketches/
>>    
>> <https://www.google.com/url?q=https://www.google.com/url?q%3Dhttps://repository.apache.org/content/groups/snapshots/org/apache/datasketches/%26source%3Dgmail-imap%26ust%3D1626465263000000%26usg%3DAOvVaw0zU9ASggdO53YTS6qMd0Bs&source=gmail-imap&ust=1626472557000000&usg=AOvVaw2YXDOOchjoKno5SYKbsC0l>
>>    during a previous attempt. This failure was cached in the local
>>    repository and resolution is not reattempted until the update interval of
>>    apache.snapshots has elapsed or updates are forced
>>
>> Hmm.  It should not be trying to locate the 2.0.0-SNAPSHOT from Apache
>> unless it can't find it locally in my .m2. But I can't create it in .m2
>> unless I do an mvn install;  and before I can do a maven install, I need to
>> successfully compile. I'm stuck in a do-loop.
>>
>> Next I tried removing all 9+ versions from both the toolchain and the
>> matrix. I get the same error.
>>
>> Lee.
>>
>>
>>
>> On Fri, Jul 9, 2021 at 12:02 PM leerho <[email protected]> wrote:
>>
>>> David,
>>> After the merge of PR#131 and doing a new clone of :
>>> Setting JAVA_HOME to JDK8
>>>
>>>    -
>>>
>>> *mvn clean compile successful compile of java 8, but fails on java 9 *
>>>
>>> *Setting JAVA_HOME to JDK11*
>>>
>>>    -
>>> *mvn clean compile *
>>>    *Reactor Summary for datasketches-memory-root 2.0.0-SNAPSHOT:*
>>>    -
>>> *[INFO] datasketches-memory-root ........................... SUCCESS [
>>>       0.912 s] [INFO] datasketches-memory-java8 ..........................
>>>       SUCCESS [  1.322 s][INFO] datasketches-memory-java8-tests
>>>       .................... SUCCESS [  0.087 s][INFO] 
>>> datasketches-memory-java9
>>>       .......................... SUCCESS [  0.602 s][INFO]
>>>       datasketches-memory-java9-tests .................... SUCCESS [  0.191
>>>       s][INFO] datasketches-memory-java11 ......................... SUCCESS 
>>> [
>>>       0.701 s][INFO] datasketches-memory ................................ 
>>> SUCCESS
>>>       [  0.182 s][INFO] datasketches-memory-tests ..........................
>>>       FAILURE [  1.201 s]*
>>>       -
>>>
>>>       [*ERROR*] Failed to execute goal
>>>       org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce
>>>       *(enforce-banned-dependencies)* on project
>>>       datasketches-memory-tests:
>>>
>>>
>>>
>>> *Execution enforce-banned-dependencies of goal
>>>       org.apache.maven.plugins:maven-enforcer-plugin:3.0.0-M3:enforce 
>>> failed:
>>>       
>>> org.apache.maven.shared.dependency.graph.DependencyGraphBuilderException:
>>>       Could not resolve following dependencies:
>>>       
>>> [org.apache.datasketches:datasketches-memory-java8-tests:jar:tests:2.0.0-SNAPSHOT
>>>       (test)]*:
>>>       Could not resolve dependencies for project
>>>       org.apache.datasketches:datasketches-memory-tests:jar:2.0.0-SNAPSHOT: 
>>> Could
>>>       not find artifact
>>>       
>>> org.apache.datasketches:datasketches-memory-java8-tests:jar:tests:2.0.0-SNAPSHOT
>>>       in apache.snapshots (
>>>       
>>> https://repository.apache.org/content/groups/snapshots/org/apache/datasketches/
>>>       
>>> <https://www.google.com/url?q=https://www.google.com/url?q%3Dhttps://repository.apache.org/content/groups/snapshots/org/apache/datasketches/%26source%3Dgmail-imap%26ust%3D1626465263000000%26usg%3DAOvVaw0zU9ASggdO53YTS6qMd0Bs&source=gmail-imap&ust=1626472557000000&usg=AOvVaw2YXDOOchjoKno5SYKbsC0l>
>>>       )
>>>
>>>
>>> I'm not sure how to resolve this. I'm wondering if I am missing a
>>> toolchain in my environment.
>>>
>>> Lee.
>>>
>>>
>>
>

Reply via email to