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

[DONE]Feature multiple repo #10

Merged
merged 2 commits into from
Oct 28, 2016
Merged

[DONE]Feature multiple repo #10

merged 2 commits into from
Oct 28, 2016

Conversation

gortium
Copy link
Contributor

@gortium gortium commented Oct 25, 2016

  • Merger en meme temps que les branch du meme nom dans les autre repo
  • Modify sara_robot.rosinstall and sara_simulation.rosinstall to fetch the develop branch (do the same when merging to master)

Fix #8

Copy link
Contributor

@gregoirefra gregoirefra left a comment

Choose a reason for hiding this comment

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

Please see my comment in the file changed

# ROS installation
echo "Installing ROS"

sudo apt install -y ros-indigo-desktop-full -y
Copy link
Contributor

Choose a reason for hiding this comment

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

This wont work since the setup will be done on a server without a GUI

# catkin_make to build all the packages
echo "Building packages"

catkin_make -DCATKIN_WHITELIST_PACKAGES="wm_arm_msgs"
Copy link
Contributor

Choose a reason for hiding this comment

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

Wasn't there a fix for this? I believe @maximest-pierre had solved the issue

@maximest-pierre
Copy link
Member

This was fix by me. We dont need the horrible hack to build.

@gortium
Copy link
Contributor Author

gortium commented Oct 25, 2016

On which branch ? i took master to start these.. merge and refix i guess?

@maximest-pierre
Copy link
Member

It suppose to be fix in master cause i just build master recently and there is not need for it

@gregoirefra
Copy link
Contributor

Merger et fixer après n'est vraiment pas une bonne pratique quand le problème a déjà été résolu. Essai de merge master dans ta branch en premier pour bien voir les changements

@gortium
Copy link
Contributor Author

gortium commented Oct 25, 2016

Mais.. Regarde le graph dans l'onglet network.. Ma branch part du dernier commit sur master alors je comprend pas pourquoi j'aurais pas les changes..

@gregoirefra
Copy link
Contributor

@maximest-pierre peux-tu confirmer que tu as bien commit t'es changements parce qu'effectivement c'est la même chose que dans master qu'il a mais je suis quasi certain que tu avais changé cette partie

@maximest-pierre
Copy link
Member

maximest-pierre commented Oct 25, 2016

C'est merger dans master de sara_commun.

@gregoirefra
Copy link
Contributor

Euh quoi? Peux-tu commit le changement ici aussi alors?

@gortium
Copy link
Contributor Author

gortium commented Oct 25, 2016

Haaaa ok il parle du scrip de travis... Pas la meme chose... hmm faut trouver un moyen de sync les deux 0o

@gregoirefra
Copy link
Contributor

Après validation il n'est pas necessaire de faire un catkin_make avec whitelist mais seulement un catkin_make et tout fonctionne sauf que présentement le script me donne une erreur que catkin_make n'est pas trouvé dans l'environement, toute fois en testant j'ai trouvé que si tu source ton ~/.bashrc et ensuite catkin_make, le tout dans un autre script tu es correct et ca build sans problème

@gortium
Copy link
Contributor Author

gortium commented Oct 26, 2016

Hmm mais le truc c'est que c'est pas des changes/bugfix qui devrait avoir rapport avec ma branch non?

@gortium gortium merged commit 09be29f into develop Oct 28, 2016
@gortium gortium deleted the feature_multipleRepo branch October 28, 2016 04:34
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