Skip to content

[10.0][FIX] Move DB libs to travis file - #713

Merged
pedrobaeza merged 2 commits into
OCA:10.0from
LasLabs:hotfix/10.0/base_external_dbsource-requires
Jan 25, 2017
Merged

pedrobaeza merged 2 commits into
OCA:10.0from
LasLabs:hotfix/10.0/base_external_dbsource-requires

Conversation

@lasley

@lasley lasley commented Jan 24, 2017

Copy link
Copy Markdown
Contributor

This is a stop gap to fix build while we determine how to accomplish OCA/maintainer-quality-tools#427

  • Move libraries that require development headers to travis file to band-aid build

* Move libraries that require development headers to travis file to band-aid build
@lasley lasley added this to the 10.0 milestone Jan 24, 2017
@lasley

lasley commented Jan 24, 2017

Copy link
Copy Markdown
Contributor Author

Hrmmm well that build failure was unexpected. It almost seems like the apt packages are being skipped in lint, but I don't see how that could be possible.

@lasley

lasley commented Jan 24, 2017

Copy link
Copy Markdown
Contributor Author

I added a switch to not install during lint checks. Not the prettiest solution, but I believe we're good for review here.

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good for me

@pedrobaeza
pedrobaeza requested a review from moylop260 January 24, 2017 22:38
@pedrobaeza

Copy link
Copy Markdown
Member

Let's merge this one for solving the issues that are raising all across 10.0

@pedrobaeza
pedrobaeza merged commit a8df2ca into OCA:10.0 Jan 25, 2017
@lasley
lasley deleted the hotfix/10.0/base_external_dbsource-requires branch January 25, 2017 17:21
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this pull request Nov 20, 2024
Syncing from upstream OCA/server-tools (8.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants