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

Add message for Seq remove and add group functions #477

Merged
merged 1 commit into from
Feb 13, 2015

Conversation

snozawa
Copy link
Contributor

@snozawa snozawa commented Feb 11, 2015

Seqのremove&add Group関数が呼ばれた時にprint文がでるようにしました.
よろしくお願いいたします.

@k-okada
Copy link
Contributor

k-okada commented Feb 11, 2015

Refer to this link for build results (access rights to CI server needed):
http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080/job/hrpsys-qnx/1755/

Build Log
last 10 lines

[...truncated 5368 lines...]
    at org.jenkinsci.plugins.ghprb.GhprbRepository.createCommitStatus(GhprbRepository.java:122)
    at org.jenkinsci.plugins.ghprb.GhprbBuilds.onCompleted(GhprbBuilds.java:118)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:27)
    at org.jenkinsci.plugins.ghprb.GhprbBuildListener.onCompleted(GhprbBuildListener.java:12)
    at hudson.model.listeners.RunListener.fireCompleted(RunListener.java:199)
    at hudson.model.Run.execute(Run.java:1769)
    at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
    at hudson.model.ResourceController.execute(ResourceController.java:89)
    at hudson.model.Executor.run(Executor.java:240)

Test FAILed.

@snozawa
Copy link
Contributor Author

snozawa commented Feb 13, 2015

@k-okadaさん
https://travis-ci.org/fkanehiro/hrpsys-base/jobs/50344588
のように、おそらく
start-jsk/rtmros_common#652 (comment)
に関連したエラーがapt-get install ros-hydro-hironx-ros-bridge時にでていますが、
もう少しまつ必要がありますでしょうか。

@k-okada
Copy link
Contributor

k-okada commented Feb 13, 2015

はい.Hydro版をリリースするときに
https://github.com/tork-a/openrtm_aist-release/blob/patches/release/hydro/openrtm_aist/0002-Importing-patches-from-indigo-release-branch.patch
のパッチをいれることを忘れていました.さっき直したので治ると思いますが,急いでいればマージしてもらうのでいいと思います.
(早くオフィシャルに入れてほしいものですね)

◉ Kei Okada

On Fri, Feb 13, 2015 at 5:23 PM, Shunichi Nozawa [email protected]
wrote:

@k-okada https://github.com/k-okadaさん
https://travis-ci.org/fkanehiro/hrpsys-base/jobs/50344588
のように、おそらく
start-jsk/rtmros_common#652 (comment)
start-jsk/rtmros_common#652 (comment)
に関連したエラーがapt-get install ros-hydro-hironx-ros-bridge時にでていますが、
もう少しまつ必要がありますでしょうか。


Reply to this email directly or view it on GitHub
#477 (comment).

@snozawa
Copy link
Contributor Author

snozawa commented Feb 13, 2015

わかりました.ありがとうございます.
今いくつか出しているPRで,チェックはさすがに大丈夫そうかな,と思うものは
mergeをお願いしていこうかと思います.

@fkanehiro さん
本PRはprint文を追加してるだけのものであり,OKそうに思いますので
travisのfailを無視して今回はmergeをお願いできますでしょうか.

よろしくお願いいたします.

fkanehiro added a commit that referenced this pull request Feb 13, 2015
Add message for Seq remove and add group functions
@fkanehiro fkanehiro merged commit 4cdc669 into fkanehiro:master Feb 13, 2015
@snozawa snozawa deleted the add_messge_seq_group_funcs branch February 26, 2015 16:44
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 this pull request may close these issues.

3 participants