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

    mammoth-pharmacist-54215

    04/29/2016, 10:19 PM
    safer to run against both branches wrt builds.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:20 PM
    especially as the two branches diverge
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:20 PM
    right now they are pretty close.
  • w

    witty-crayon-22786

    04/29/2016, 10:20 PM
    @mammoth-pharmacist-54215: needs clarification in the docs maybe?
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:20 PM
    yeah, probably
  • w

    witty-crayon-22786

    04/29/2016, 10:21 PM
    my feeling would be that things should hang out in master for a while before being cherry picked to the branch
  • w

    witty-crayon-22786

    04/29/2016, 10:21 PM
    but don't know how that is manageable.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:21 PM
    If @fancy-queen-20734 is up for it we can work through the 1.0.1 release and fix up the rough spots in the docs as he hits them.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:21 PM
    I think it depends on the fix.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:22 PM
    the greater the complexity the more bake time on master I’d like to see.
  • f

    fancy-queen-20734

    04/29/2016, 10:22 PM
    yeah, like https://rbcommons.com/s/twitter/r/3786/ is by inspection pretty unlikely to cause problems if cherry-picked, but other changes may not be so clear cut
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:23 PM
    exactly as long as that has good test coverage I’d feel pretty comfortable about it.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:25 PM
    also want to update the docs to mention the http auth for release.
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:26 PM
    or land @bored-art-40741 ’s change so that it uses ssh
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:41 PM
    @fancy-queen-20734: is 3790 one of your perf fixes you want in 1.0.1?
  • f

    fancy-queen-20734

    04/29/2016, 10:42 PM
    yep
  • m

    mammoth-pharmacist-54215

    04/29/2016, 10:42 PM
    would you mind adding patrick to it?
  • f

    fancy-queen-20734

    04/29/2016, 10:44 PM
    np
  • c

    curved-vr-99634

    04/29/2016, 10:46 PM
    hi, so at some point 
    pants go-env
    stopped working as expected for me. i’m not quite sure when it changed though, since i had a break from coding in go for awhile
  • c

    curved-vr-99634

    04/29/2016, 10:46 PM
    not quite sure if it changed or if i’m doing something wrong, but it’s not finding third party dependencies
  • c

    curved-vr-99634

    04/29/2016, 10:47 PM
    i’m using it to start sublime and working with some golang sublime plugins that no longer work correctly
  • c

    curved-vr-99634

    04/29/2016, 10:49 PM
    here’s an example of the type of error i get when i try to use functionality like go oracle’s 
    show callers
  • c

    curved-vr-99634

    04/29/2016, 10:49 PM
    Copy code
    /Users/kellyellis/src/mono/golang/src/hopper/data/relationqueries.go:11:2: could not import <http://github.com/aws/aws-sdk-go/service/dynamodb|github.com/aws/aws-sdk-go/service/dynamodb> (cannot find package "<http://github.com/aws/aws-sdk-go/service/dynamodb|github.com/aws/aws-sdk-go/service/dynamodb>" in any of:
	/Users/kellyellis/.cache/pants/bin/go/mac/10.10/1.5.3/unpacked/go/src/github.com/aws/aws-sdk-go/service/dynamodb (from $GOROOT)
	/Users/kellyellis/code/golang/src/github.com/aws/aws-sdk-go/service/dynamodb (from $GOPATH)
	/Users/kellyellis/src/mono/golang/src/github.com/aws/aws-sdk-go/service/dynamodb
	/Users/kellyellis/src/mono/3rdparty/golang/src/github.com/aws/aws-sdk-go/service/dynamodb)
  • e

    enough-analyst-54434

    04/29/2016, 10:51 PM
    As a sanity check, (example from pants repo itself):
    Copy code
    $ ./pants clean-all go-env contrib/go/examples/:: -- echo \$GOPATH | tr : '\n'
...
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.libC.libC
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.gorilla.context.context
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.robertkrimen.otto.otto
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.kr.pretty.pretty
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.libE.libE
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.bitly.go-simplejson.go-simplejson
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.gopkg.in.fsnotify.v0.fsnotify.v0
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.hello.hello
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.libA.libA
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.cbroglie.mapstructure.mapstructure
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.kr.text.text
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.libD.libD
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.bmizerany.assert.assert
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.cgo.cgo
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.gorilla.mux.mux
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.gopkg.in.check.v1.check.v1
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.server.server
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.3rdparty.go.github.com.AdRoll.goamz.dynamodb
/home/jsirois/dev/pantsbuild/jsirois-pants/.pants.d/go-env/go-env/contrib.go.examples.src.go.libB.libB
  • e

    enough-analyst-54434

    04/29/2016, 10:51 PM
    Can you do something similar and see if it looks wrong in the way your error above complains?
  • e

    enough-analyst-54434

    04/29/2016, 10:51 PM
    @curved-vr-99634: you all are on what pants version by the way?
  • e

    enough-analyst-54434

    04/29/2016, 10:52 PM
    ./pants --version
  • c

    curved-vr-99634

    04/29/2016, 10:52 PM
    1.0.0-rc3
  • e

    enough-analyst-54434

    04/29/2016, 10:52 PM
    k
  • c

    curved-vr-99634

    04/29/2016, 10:52 PM
    sec will run the other cmd
1...122512261227...2420Latest