Skip to content
This repository has been archived by the owner on Apr 24, 2023. It is now read-only.

Fix build dependency orders related to igvc_msgs #40

Closed
barulicm opened this issue Apr 16, 2015 · 4 comments · Fixed by #77
Closed

Fix build dependency orders related to igvc_msgs #40

barulicm opened this issue Apr 16, 2015 · 4 comments · Fixed by #77
Labels
type ➤ bug Bugs to existing features that need to be fixed. type ➤ structural Structural issues of our code
Milestone
End of Fall 2016

Comments

@barulicm
Copy link
Contributor

On first build, some nodes in IGVC get built before igvc_msgs finishes, causing them to error out. Currently, all builds after the first one work fine, but we should still probably fix this.
@barulicm barulicm added the type ➤ bug Bugs to existing features that need to be fixed. label Apr 16, 2015
@alchaussee alchaussee mentioned this issue Sep 24, 2015
Closed
@alchaussee
Copy link
Contributor

Can run "catkin_make igvc_msgs_gencpp" to build the messages then catkin_make will work

@jgkamat
Copy link
Contributor

jgkamat commented Sep 24, 2015

Running make without -j2 works, but I can't find how to turn that off in catkin. Also...
deps

This is a picture...

@barulicm
Copy link
Contributor Author

That's a work of art

jgkamat added a commit to jgkamat/igvc-software that referenced this issue Dec 1, 2015
@jgkamat
Try to work around the igvc_msgs issue (RoboJackets#40)
c48ddde
@jgkamat jgkamat mentioned this issue Dec 1, 2015
Merged
4 tasks
@jgkamat
Copy link
Contributor

jgkamat commented Jan 21, 2016

Whoops

@jgkamat jgkamat reopened this Jan 21, 2016
@rmkeezer rmkeezer added the type ➤ structural Structural issues of our code label Sep 14, 2016
dpattison3 added a commit to dpattison3/igvc-software that referenced this issue Oct 12, 2016
@dpattison3
fixed build dependencies on igvc_msgs package - could certainly be a …
da3eaa7 
…cleaner solution, but the project should be able to be built first try - closes issue RoboJackets#40
dpattison3 added a commit to dpattison3/igvc-software that referenced this issue Oct 16, 2016
@dpattison3
fixed build dependencies on igvc_msgs package - could certainly be a …
e9e2778 
…cleaner solution, but the project should be able to be built first try - closes issue RoboJackets#40
jgkamat added a commit that referenced this issue Oct 17, 2016
@jgkamat
Stop building igvc_msgs before all
4520ee5 
See #40
jgkamat added a commit that referenced this issue Oct 17, 2016
@jgkamat
Stop building igvc_msgs before all
a2e6956 
See #40
@jgkamat jgkamat mentioned this issue Oct 18, 2016
Merged
@rmkeezer rmkeezer added this to the End of Fall 2016 milestone Oct 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
type ➤ bug Bugs to existing features that need to be fixed. type ➤ structural Structural issues of our code
Projects
None yet
Milestone
End of Fall 2016
Development

Successfully merging a pull request may close this issue.

Add CI To igvc-software RoboJackets/igvc-software
5 participants
@barulicm @alchaussee @jgkamat @rmkeezer @dpattison3