Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Start fixing appveyor #2042

Closed
wants to merge 57 commits into from
Closed

Conversation

dignifiedquire
Copy link
Member

No description provided.

whyrusleeping and others added 9 commits November 16, 2015 09:17
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Friedel Ziegelmayer <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: Friedel Ziegelmayer <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: Friedel Ziegelmayer <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@jbenet jbenet added the backlog label Dec 5, 2015
@jbenet
Copy link
Member

jbenet commented Dec 6, 2015

@jbenet
Copy link
Member

jbenet commented Dec 6, 2015

also, Feodor said

Both MinGW and MSYS are installed on unstable image.

@dignifiedquire
Copy link
Member Author

Progress, now make is running, but go install fails but need some more insight on that from go pros ;) https://ci.appveyor.com/project/ipfsbot/test/build/33

cc @whyrusleeping @jbenet

@jbenet
Copy link
Member

jbenet commented Dec 6, 2015

looks like the project is cloned to:

git clone -q https://github.com/ipfs/go-ipfs.git C:\projects\test

but that's not in the gopath, and it's looking for the packages in the proper gopath:

daemon.go:14:2: cannot find package "github.com/ipfs/go-ipfs/Godeps/_workspace/src/github.com/codahale/metrics/runtime" in any of:
    c:\go\src\github.com\ipfs\go-ipfs\Godeps\_workspace\src\github.com\codahale\metrics\runtime (from $GOROOT)
    c:\gopath\src\github.com\ipfs\go-ipfs\Godeps\_workspace\src\github.com\codahale\metrics\runtime (from $GOPATH)

I would try:

mkdir -p c:\go\src\github.com\ipfs
mv C:\projects\test c:\go\src\github.com\ipfs\go-ipfs
cd c:\go\src\github.com\ipfs\go-ipfs
# carry on

i dont think symlinks will work

@dignifiedquire
Copy link
Member Author

Thanks @jbenet I changed the target clone directory to what you suggested and the go install is now working as expected. Sadly it now fails because it tries to install bazil.org/fuse which has no support on windows, so we need a way that go install does not even tries to install it as far as I understand.

build_script:
- make install
- mingw32-make install
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the doc, it appears this should have been mingw32-make nofuse in windows.
An os detect to add the nofuse flag could have been incorporated into the makefile.

@rht
Copy link
Contributor

rht commented Dec 6, 2015

For faster build,

cache:
- C:\ProgramData\chocolatey\bin -> appveyor.yml
- C:\ProgramData\chocolatey\lib -> appveyor.yml

can be added [1].

[1] http://www.appveyor.com/docs/build-cache

@dignifiedquire
Copy link
Member Author

@rht actually we don't need to install anything with chocolatey I realised, mingw and go 1.5.1 are already preinstalled :)

@dignifiedquire
Copy link
Member Author

@rht I'm not able to pass the nofuse flag properly through to the tests. Any ideas how to fix that?

@rht
Copy link
Contributor

rht commented Dec 6, 2015

What about adding satisfied_prereq: EXPENSIVE NOFUSE to the env manually?

@rht
Copy link
Contributor

rht commented Dec 6, 2015

I mean satisfied_prereq: EXPENSIVE. FUSE seems to be opt-in rather than default.

Since the TEST_EXPENSIVE flag is used to set the satisfied_prereq flag (https://github.com/ipfs/go-ipfs/blob/9066b12d33cd67e083c15c58767c5a629c5165e2/test/sharness/lib/test-lib.sh#L42-L43).

@rht
Copy link
Contributor

rht commented Dec 6, 2015

test_set_prereq.

If it doesn't work, the likely cause is due to satisfied_prereq being overridden after installing sharness

test_script:
- make $TEST_SUITE
- make test_sharness_expensive
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually, since go builds just fine, go test -tags nofuse ./... can be added before running the sharness test. However, expect to see lots of goroutine errs.

@rht
Copy link
Contributor

rht commented Dec 6, 2015

@dignifiedquire, so it appears windows shell uses a different syntax to set the env var[1].
The sharness lib assumes it is run in a POSIX shell...

[1] #1982 (comment)

@dignifiedquire
Copy link
Member Author

yeah I was afraid of that, looks like most of the test setup is posix specific :( I will try to look if I can use cygwin or sth like that to emulate it, otherwise this will be really tricky

@dignifiedquire
Copy link
Member Author

@rht okay switched to using cygwin bash, which should support environment variables in posix style, but for some reason it's still trying to run the fuse tests :(

@rht
Copy link
Contributor

rht commented Dec 6, 2015

TEST_NO_FUSE is to run the test without fuse, but the binary is still built with fuse, which then fails.

Re-enabling GOFLAGS: -tags nofuse should do.

@dignifiedquire
Copy link
Member Author

@rht still failing :(

Jonathan Dahan and others added 2 commits December 17, 2015 02:02
License: MIT
Signed-off-by: Jonathan Dahan <jonathan@jonathan.is>
Have install.sh put the full path to ipfs binary if detected
@ghost ghost added the kind/test Testing work label Dec 22, 2015
djdv and others added 23 commits December 22, 2015 10:59
License: MIT
Signed-off-by: Dominic Della Valle <ddvpublic@gmail.com>
Fix path parser in add command
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
disable auto repo gc tests temporarily
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
License: MIT
Signed-off-by: Adam Gashlin <agashlin@gmail.com>
fix typo in AddrFilters error message
License: MIT
Signed-off-by: rht <rhtbot@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
License: MIT
Signed-off-by: dignifiedquire <dignifiedquire@gmail.com>
@GitCop
Copy link

GitCop commented Dec 29, 2015

There were the following issues with your Pull Request

  • Commit: 243a158
    • Invalid signoff. Commit message must end with
      License: MIT
      Signed-off-by: .* <.*>

Guidelines and a script are available to help. Your feedback on GitCop is welcome on this issue.


This message was auto-generated by https://gitcop.com

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/test Testing work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants