Hi Martin,
On 04/14/2014 08:59 AM, Konrad, Martin wrote:
> did you notice that I pushed our putlog patch again? This time the
> commit ids should be correct. I just realized that I didn't get an
> automatic email on core talk...
>
> https://code.launchpad.net/~info-martin-konrad/epics-gateway/putlog/+merge/213921
I saw the merge proposal (you might not get copies of your own proposals
if your core-talk subscription is configured not to send you copies of
your own messages and Launchpad and core-talk both know you by the same
email address), sorry I haven't responded yet.
Your changes look reasonable from the perspective of someone who doesn't
know the application internals at all. I haven't tried building the
result or looked at what it's actually trying to accomplish in detail
though. I would prefer if someone like Ralph who knows the code better
than I could take a look too, but if he can't spare the time I will
probably approve merging this anyway.
One minor comment: I'd suggest renaming the "ALLOW_PUTLOG" macro to
"WITH_PUTLOG" which I think reads slightly better. This wording seems to
be commonly used outside the EPICS community. If you do make that change
you won't need to re-propose the merge; Launchpad will see the change on
the branch and include the update automatically.
I'm wondering if we should change the way we build the PV Gateway to
make it a stand-alone module instead of depending on extensionsConfig.
If they're anything like the APS sites may want to build this statically
and copy the binary to the specific gateway machines anyway, so building
this into extensions doesn't make a lot of sense.
Thanks,
- Andrew
--
Advertising may be described as the science of arresting the human
intelligence long enough to get money from it. -- Stephen Leacock
- Navigate by Date:
- Prev:
Jenkins build is back to normal : epics-base-3.14-vx69 #13 APS Jenkins
- Next:
Re: putlog patch for gateway Konrad, Martin
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
<2014>
2015
2016
2017
2018
2019
2020
2021
2022
2023
2024
2025
- Navigate by Thread:
- Prev:
Jenkins build is back to normal : epics-base-3.14-linux32 #18 APS Jenkins
- Next:
Re: putlog patch for gateway Konrad, Martin
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
<2014>
2015
2016
2017
2018
2019
2020
2021
2022
2023
2024
2025
|