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

mcdb and libhepml dependency #14728

Closed
smuzaffar opened this issue Jun 1, 2016 · 10 comments
Closed

mcdb and libhepml dependency #14728

smuzaffar opened this issue Jun 1, 2016 · 10 comments

Comments

@smuzaffar
Copy link
Contributor

Looks like mcdb and libhepml are not actually used in cmssw. Only reference to these tools are
https://github.com/cms-sw/cmssw/blob/CMSSW_8_1_X/GeneratorInterface/LHEInterface/BuildFile.xml
https://github.com/cms-sw/cmssw/blob/CMSSW_8_1_X/GeneratorInterface/LHEInterface/plugins/BuildFile.xml
https://github.com/cms-sw/cmssw/blob/CMSSW_8_1_X/GeneratorInterface/MCatNLOInterface/plugins/BuildFile.xml

but these can be removed without breaking the build.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 1, 2016

A new Issue was created by @smuzaffar (Malik Shahzad Muzaffar).

@davidlange6, @smuzaffar, @Degano, @davidlt, @Dr15Jones can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@smuzaffar
Copy link
Contributor Author

assign generators
can you please check this? If not needed then we can drop these tools from cmssw externals

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 1, 2016

New categories assigned: generators

@thuer,@covarell,@perrozzi,@govoni you have been requested to review this Pull request/Issue and eventually sign? Thanks

@smuzaffar
Copy link
Contributor Author

@thuer,@covarell,@perrozzi,@govoni any issues removing this tool for cmssw?

@smuzaffar
Copy link
Contributor Author

ping?

@govoni
Copy link
Contributor

govoni commented Jul 18, 2016

I think it's ok

@smuzaffar
Copy link
Contributor Author

assign externals

@cmsbuild
Copy link
Contributor

New categories assigned: externals

@smuzaffar,@gudrutis,@mrodozov you have been requested to review this Pull request/Issue and eventually sign? Thanks

@smuzaffar
Copy link
Contributor Author

cms-sw/cmsdist#5886 and #30042 remove these dependencies.

@smuzaffar
Copy link
Contributor Author

+externals

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants