Skip to content

Conversation

betzw
Copy link
Contributor

@betzw betzw commented Mar 15, 2016

in order to align with 'mbed-drivers' usage of this type of macros.

See also PR ARMmbed/mbed-hal-st-stm32f401re#8

@betzw
Copy link
Contributor Author

betzw commented Mar 16, 2016

@0xc0170 is there a way to inform Jenkins CI that a certain PR depends on another one in a different repo to get also merged in order to pass the tests?

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 16, 2016

@0xc0170 is there a way to inform Jenkins CI that a certain PR depends on another one in a different repo to get also merged in order to pass the tests?

No, currently not. For the PR like this, modules_ PR would get merged first, jenkins restarted afterwards for this PR.
cc @PrzemekWirkus, the feature might be useful :)

@betzw
Copy link
Contributor Author

betzw commented Mar 17, 2016

But merging first PR ARMmbed/mbed-hal-st-stm32f401re#8 would lead (temporarily) to a situation in which the actual master branches of the two repos mbed-hal-st-stm32f4 and mbed-hal-st-stm32f401re are not consistent among each other (i.e. do not compile together).
Furthermore changing these macro names must be done for all STM32F4xx platforms, so the problem spans even over more than two repos!

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 17, 2016

@betzw You could make it backward compatible? I haven't checked the current tree dependencies, would it help if here you would do

#ifndef MODULES_SIZE_SPI
#define MODULES_SIZE_SPI MODULE_SIZE_SPI
#endif

Or you can use more narrow dependencies if required.

@betzw
Copy link
Contributor Author

betzw commented Mar 18, 2016

Good idea!
But I think once all F4 platforms have adjusted their macro definitions we should remove the

#ifndef ... 
#define ... 
#endif

again.
What do you think?

@0xc0170
Copy link
Contributor

0xc0170 commented Mar 18, 2016

// Temporary workaround to pass Jenkins CI

It's not for jenkins but for anyone who uses this module. Yes, seems fine now.

To illustrate what I meant with narrowing down dependencies, might help you in the future. Lets say stm32f4xxx is as target dependency specified ^1.0.5. Once you update that module (for instance to 1.0.6), you can update the module dependency to ^1.0.6 and remove the changes (trick with ifdef above) for this update.

@betzw
Copy link
Contributor Author

betzw commented Mar 18, 2016

Thanks a lot for the hint on dependencies.
I'll give it a try.

@bcostm bcostm merged commit 330402e into ARMmbed:master Apr 6, 2016
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