Skip to content

ci/after_script.sh missing? #3857

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

Closed
gliptak opened this issue Jun 12, 2013 · 16 comments · Fixed by #3868
Closed

ci/after_script.sh missing? #3857

gliptak opened this issue Jun 12, 2013 · 16 comments · Fixed by #3868

Comments

@gliptak
Copy link
Contributor

gliptak commented Jun 12, 2013

https://travis-ci.org/gliptak/pandas/jobs/8002655

$ ci/print_versions.py
$ ci/after_script.sh
/home/travis/build.sh: line 208: ci/after_script.sh: No such file or directory
Done. Your build exited with 1.

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

that should be removed from travis since there's no after_script.sh anywhere in pandas...pr?

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

nevermind i did it

@jreback
Copy link
Contributor

jreback commented Jun 12, 2013

what happend to the after script? was it ever there? what runs it?

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

was in .travis.yml in the after_script clause. was being run by travis after ci/print_versions.py

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

oh the script itself i dunno i can check the git history if uwant

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

git log -p -S 'after_script' will show it

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

doesn't look like that script was ever there...

@jreback
Copy link
Contributor

jreback commented Jun 12, 2013

so just called in ci/script.sh?

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

no travis calls it after ci/print_versions.py

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

do this

git log -S 'after_script' --pretty=format:'%h' | while read p; do
    git diff-tree --name-only $p HEAD | grep 'after_script.sh'
done

should not show anything. which means none of the cchanges mentioning after_script actually added a file with the name after_script.sh

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

that might take a while since it has to search the entire history cuz it uses pickaxe

@jreback
Copy link
Contributor

jreback commented Jun 12, 2013

so travis calls it by default....then I guess we should have one (that does nothing), right?

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

Only called if it's in the config... Not called by default

@jreback
Copy link
Contributor

jreback commented Jun 12, 2013

so it must be in the config then?

@cpcloud
Copy link
Member

cpcloud commented Jun 12, 2013

hence my change here i just removed it...

@jreback
Copy link
Contributor

jreback commented Jun 12, 2013

ok...duh!....go ahead

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

Successfully merging a pull request may close this issue.

3 participants