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.

Sign up for GitHub

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

Jump to bottom

New graph resolution detects build_requires transitive conflicts #4514

Merged
merged 154 commits into from
Mar 6, 2019
Merged

New graph resolution detects build_requires transitive conflicts #4514

merged 154 commits into from
Mar 6, 2019

Conversation

memsharded
Copy link
Member

@memsharded memsharded commented Feb 13, 2019 •
edited
Loading

Changelog: Bugfix: Build-requires expand over the closure of the package they apply to, so they can create conflicts too. Previously, those conflicts were silently skipped, and builds would use an undetermined version and configuration of dependencies.
Docs: omit

@tags: slow
@revisions: 1

Close #3949

lasote added 30 commits December 14, 2018 10:57
@lasote
WIP
50fc6c4
@lasote
Merged develop
973f737
@lasote
layout of dev
9a66eb0
@lasote
Merge remote-tracking branch 'origin/develop' into feature/revisions_…
2b4d306 
…list_endpoints
@lasote
wip
386c089
@lasote
Removed versions
46acbd6
@lasote
Merge branch 'feature/fix_server_launcher' into feature/revisions_lis…
d0ab9a3 
…t_endpoints
@lasote
working but not tested
fca987f
@lasote
local
06b1644
@lasote
Merged develop
94151fb
@lasote
Local, package failing because of none timestamp in PREV
5ea2b7a
@lasote
WIP
e125e76
@lasote
Merged develop
e4c088f
@lasote
search tests ok
7d9d2d2
@lasote
Fixing tests
1e51cac
@lasote
Fixing tests and small refactor
2daab03
@lasote
renamed module
33634ad
@lasote
review
fb71f45
@lasote
ISO in client, reviewed
b841fe9
@lasote
Fixing tests
c6cd7bd
@lasote
Merged develop
5ee5bc1
@lasote
Fixing tests
8db4701
@lasote
New tests about metadata being updated
f42c21b
@lasote
Review
a2b0d0b
@lasote
Time cleared on export/package
ca3068f
@lasote
Fix revisions list
fb21d63
@lasote
fixed test
6aa300d
@lasote
Unused import
b80b83c
@lasote
Merge remote-tracking branch 'origin/develop' into feature/remove_com…
0e6b918 
…patibility_mode
@lasote
WIP
8c302b0
@memsharded memsharded changed the title WIP: CI: Feature/new graph builder Feature/new graph builder Feb 26, 2019
@memsharded memsharded changed the title Feature/new graph builder New graph resolution and package_id default modes Feb 26, 2019
@memsharded memsharded assigned lasote and unassigned memsharded Feb 26, 2019
@memsharded memsharded mentioned this pull request Feb 26, 2019
Closed
@ghost ghost assigned memsharded Feb 27, 2019
@memsharded memsharded added this to the 1.13 milestone Mar 1, 2019
@memsharded memsharded changed the title New graph resolution and package_id default modes New graph resolution detects build_requires transitive conflicts Mar 1, 2019
@lasote lasote merged commit 3b74f2a into conan-io:develop Mar 6, 2019
@ghost ghost removed the stage: review label Mar 6, 2019
@memsharded memsharded deleted the feature/new_graph_builder branch March 6, 2019 12:32
@brianembry brianembry mentioned this pull request Mar 11, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@lasote lasote

@memsharded memsharded

Labels
None yet
Projects
None yet
Milestone
1.13
Development

Successfully merging this pull request may close these issues.

Report warnings for conflicting versions of build_requires and regular requires
2 participants
@memsharded @lasote