Summary: | =sci-physics/geant-4.10.00 - problems with SetNavigatorForTracking / geant-vmc compatibility | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Oliver Freyermuth <o.freyermuth> |
Component: | Current packages | Assignee: | Gentoo Science Physics related packages <sci-physics> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | o.freyermuth |
Priority: | Normal | Keywords: | PATCH |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Patch for SetNavigatorForTracking method (credits Ivana Hrivnacova) |
Description
Oliver Freyermuth
2014-01-01 15:35:52 UTC
Created attachment 366692 [details, diff] Patch for SetNavigatorForTracking method (credits Ivana Hrivnacova) Created from the changed files attached to: http://bugzilla-geant4.kek.jp/show_bug.cgi?id=1537 Hey, Oliver. Happy new year! Thanks for the patch. It feels not cool to comment out the code block in Lines 59-64 and Lines 70-75, though. Hey Benda, a happy new year to you, too! I know, as the method is completely reimplemented in the .cc file (including comments), one could of course also completely delete those lines. The block which is commented out contains only documentation-comments (describing the method) and actually only a single line of code (as is also visible in the .patch-file), all of which is replicated by the patch in the .cc file. As upstream's repo is private I sadly cannot see which exact route they have taken (inlining/implementation in header does not work anymore as is described in the upstream bug report). My best guess is they removed the lines in the header file completely. I could of course also recreate the patch with those lines completely removed, would that be nicer? Hi, Oliver, Thanks. I just throw some random comments to see if you really care about this bug, otherwise I'll wait for the next release. As a temporary bug fix, the not-so-good code style is okay. It's going to disappear anyway. Committed. You are always welcome for more patches. And thanks again! + 06 Jan 2014; Benda Xu <heroxbd@gentoo.org> + +files/geant-4.10.0-TrackingNavigator.patch, geant-4.10.00.ebuild: + backport upstream SetNavigatorForTracking inline function fix, credits Oliver + Freyermuth for reporting bug 496678 |