Geometry/CommonTopologies/interface/GeometryAligner.h should be moved

[Dr15Jones]

DataFormats packages are allowed to depend on Geometry/CommonTopologies. The header GeometryAligner.h has dependencies on CondFormats and is the only reason Geometry/CommonTopologies (and therefore a number of DataFormats) package depends on CondFormats.

[Dr15Jones]

assign Geometry/CommonTopologies

[cmsbuild]

New categories assigned: geometry

@Dr15Jones,@civanch,@bsunanda,@makortel,@mdhildreth you have been requested to review this Pull request/Issue and eventually sign? Thanks

[cmsbuild]

cms-bot internal usage

[cmsbuild]

A new Issue was created by @Dr15Jones.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[bsunanda]

Either it could be moved to Alignment/CommonAlignment or moved to a new pckage CommonAlignment in Geometry

[bsunanda]

Most likely the second solution will be better. Asking for a new package Geometry/CommonAlignment. Please suggest

[bsunanda]

@Dr15Jones @makortel @civanch Please look at my comment and suggest

[makortel]

After poking a bit of what currently depends on GeometryAligner.h and the existing dependencies of various Geometry packages, I agree a new package would make most sense. I let others comment if CommonAlignment would be a good name (my feeling is "not necessarily", but I don't have anything significantly better to suggest, except maybe Geometry/GeometryAligner or something like that).

[makortel]

+geometry

GeometryAligner.h was moved in #45628

[makortel]

@cmsbuild, please close

[cmsbuild]

This issue is fully signed and ready to be closed.