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

Version available in c #158

Merged
merged 1 commit into from
Jun 9, 2015
Merged

Conversation

mfine
Copy link
Contributor

@mfine mfine commented Jun 9, 2015

This is what I got for getting a c header version. Not great (you have to choose the version ahead of time, which might change by the time you finally bring it in). But oh well.

/cc @mookerji @cbeighley

@swiftnav-jenkins
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://zazu.ci.swiftnav.com/job/libsbp_Pull_Requests/130/
Test PASSed.

@mookerji
Copy link
Contributor

mookerji commented Jun 9, 2015

What do you mean by "change?"

@mfine
Copy link
Contributor Author

mfine commented Jun 9, 2015

Since the version is generated, there can be races to get versions in (I do v16 you do v16, then we merge, etc.). Also we need to manually upkeep this as versions change. Not the end of the world, but something to keep an idea about. Maybe there's a better way to do this?

@mookerji
Copy link
Contributor

mookerji commented Jun 9, 2015

lgtm

mfine added a commit that referenced this pull request Jun 9, 2015
@mfine mfine merged commit 52dfa4a into swift-nav:master Jun 9, 2015
@mfine mfine deleted the mfine-c-version-header branch June 9, 2015 17:23
silverjam pushed a commit that referenced this pull request Jun 13, 2023
…cific (#158) (#1341)

Automated PR by Jenkins. If CI has passed successfully, merge away!

**cmake**  12b7f037  ->  04c7c144
- 04c7c144 : fix: make test_srcs target project specific
(swift-nav/cmake#158)


This pull request was created by
https://jenkins.ci.swift-nav.com/job/CI%20Infra/job/submodule-update/14394/.
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