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

Fail at run time if Service uses an obsolete signal. #1209

Merged
merged 1 commit into from Oct 28, 2013

Conversation

Dr15Jones
Copy link
Contributor

The threaded framework requires Services to switch to a new collection
of signals. To allow easier migration, the obsolete signals have been
replaced with a class which causes an Exception to be thrown if their
connect method is called.
Had to change one unit test which was making use of the old signals.

The threaded framework requires Services to switch to a new collection
of signals. To allow easier migration, the obsolete signals have been
replaced with a class which causes an Exception to be thrown if their
connect method is called.
Had to change one unit test which was making use of the old signals.
@Dr15Jones
Copy link
Contributor Author

+1
This will break nearly all cmsRun jobs, but that is what we want.

ktf added a commit that referenced this pull request Oct 28, 2013
Fail at run time if Service uses an obsolete signal.
@ktf ktf merged commit 6263ce7 into cms-sw:CMSSW_7_0_THREADED_X Oct 28, 2013
@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests.

EmyrClement pushed a commit to EmyrClement/cmssw that referenced this pull request Feb 21, 2024
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.

None yet

3 participants