https://pantsbuild.org/ logo
Powered by
#general
  • b

    bored-art-40741

    04/10/2016, 8:24 PM
    hm I guess you shouldn't need one here anyway
  • m

    mysterious-action-44812

    04/10/2016, 8:26 PM
    gistfile1.txt
    gistfile1.txt
  • a

    ancient-stone-50795

    04/10/2016, 8:32 PM
    @mysterious-action-44812: I downloaed the jar. Looks like the name of the resolver is: 
    ohnosequences.ivy.S3Resolver
  • m

    mysterious-action-44812

    04/10/2016, 8:32 PM
    yeah I tried that too
  • m

    mysterious-action-44812

    04/10/2016, 8:32 PM
    still same error
  • m

    mysterious-action-44812

    04/10/2016, 8:33 PM
    this one too Exception in thread "main" java.text.ParseException: failed to load settings from file/Users/nitay/code/aiq/build support/ivysettings.xml impossible to define new type: class not found: fm.sbt.S3URLResolver in [] nor Ivy classloader when I pulled in another lib
  • m

    mysterious-action-44812

    04/10/2016, 8:33 PM
    sth about the dependencies in ivy.xml not actually getting pulled
  • a

    ancient-stone-50795

    04/10/2016, 8:34 PM
    Well, that is an sbt specific class. we don't have those defined so maybe something extra needs to be added to pull that in too
  • a

    ancient-stone-50795

    04/10/2016, 8:34 PM
    Looking at the 
    pom.xml
    file, the s3Resolver depends on:
    Copy code
    <dependencies>
<dependency>
<groupId>com.amazonaws</groupId>
<artifactId>aws-java-sdk-s3</artifactId>
<version>1.10.47</version>
</dependency>
<dependency>
<groupId>org.apache.ivy</groupId>
<artifactId>ivy</artifactId>
<version>2.4.0</version>
</dependency>
</dependencies>
  • m

    mysterious-action-44812

    04/10/2016, 8:34 PM
    ye that was just me playing with it, the first one should really work
  • m

    mysterious-action-44812

    04/10/2016, 8:35 PM
    right
  • a

    ancient-stone-50795

    04/10/2016, 8:35 PM
    You may need to add those two dependencies to ivy.xml to get it to work, although I would think that ivy would find those by itself.
  • m

    mysterious-action-44812

    04/10/2016, 8:36 PM
    ye same error
  • m

    mysterious-action-44812

    04/10/2016, 8:37 PM
    at this point im thinking to build a fatjar and use that bootstrap thing
  • m

    mysterious-action-44812

    04/10/2016, 8:42 PM
    though im not even sure that will work b/c ivy resolves itself at some point right
  • m

    mysterious-action-44812

    04/10/2016, 8:43 PM
    so then it would grab the wrong one
  • w

    witty-crayon-22786

    04/10/2016, 8:53 PM
    @bored-art-40741: re the engine failure: yea, that's what used to cause the hang, but which is now exposed. Peiyu has a ticket about it; one us will look at it this week
  • b

    bored-art-40741

    04/10/2016, 8:53 PM
    Ah, okay
  • b

    bored-art-40741

    04/10/2016, 8:54 PM
    Better than a hang!
  • m

    mysterious-action-44812

    04/10/2016, 8:55 PM
    @ancient-stone-50795: how do I know that its really reading ivy.xml
  • m

    mysterious-action-44812

    04/10/2016, 9:04 PM
    when i change the deps to nonsensical stuff if doesnt fail, i dont think its actually trying to fetch anything
  • e

    enough-analyst-54434

    04/10/2016, 11:44 PM
    @mysterious-action-44812 - 2 issues here: 1: the ivy bootstrapper uses the ivysettings.xml when bootstrapping the full ivy classpath and this leads to class not found when the enhanced classpath itself contributes to the ivysettings.xml classpath, as it needs to in your case to provide a resolver 2: The resolver here is not ready-to-use by ivy: https://github.com/ohnosequences/ivy-s3-resolver/blob/master/src/main/java/ohnosequences/ivy/S3Resolver.java - Ivy needs a no-arg constructor / has an xml protocol for property population that is not followed here, so a shim will need to be created and published. Both of these are pretty small fixes. It would be good to file issues if you want these to see some action.
  • e

    enough-analyst-54434

    04/10/2016, 11:46 PM
    This is enough to get past 1:
    Copy code
    $ diff pants/ivy/bootstrapper.py.orig pants/ivy/bootstrapper.py
159c159
<                ivy_settings=self._ivy_subsystem.get_options().ivy_settings,
---
>                ivy_settings=None,
  • e

    enough-analyst-54434

    04/10/2016, 11:46 PM
    (Though thats not the right fix)
  • m

    mysterious-action-44812

    04/11/2016, 12:02 AM
    @enough-analyst-54434: thx that helps guide yeah 2 I think I can fix, 1 I’ll try your fix
  • m

    mysterious-action-44812

    04/11/2016, 12:04 AM
    i’m new to python/pants but have done a lot of java so that stuff i’m at least more familiar roaming around
  • m

    mysterious-action-44812

    04/11/2016, 12:05 AM
    @enough-analyst-54434: q how do I use a locally edited pants?
  • e

    enough-analyst-54434

    04/11/2016, 12:25 AM
    You really don't, I was just pointing out a Pants fix where a different ivysettings.xml can be specified for bootstrap phase 2, is what is needed.
  • m

    mysterious-action-44812

    04/11/2016, 12:27 AM
    gotcha we can add an options for separate one
  • m

    mysterious-action-44812

    04/11/2016, 12:29 AM
    @enough-analyst-54434: hmm now i get this - https://gist.github.com/nitay/3d0606d98a742eb44c09a1b2a45812e0
1...114711481149...2420Latest