BIND 10 #2899: move interprocess_sync under lib/log or make it a separate private lib
BIND 10 Development
do-not-reply at isc.org
Fri May 10 07:40:47 UTC 2013
#2899: move interprocess_sync under lib/log or make it a separate private lib
-------------------------------------+-------------------------------------
Reporter: jinmei | Owner:
Type: defect | jinmei
Priority: medium | Status:
Component: build system | reviewing
Keywords: | Milestone:
Sensitive: 0 | Sprint-20130514
Sub-Project: Core | Resolution:
Estimated Difficulty: 3 | CVSS Scoring:
Total Hours: 1.29 | Defect Severity: N/A
| Feature Depending on Ticket:
| Add Hours to Ticket: 0
| Internal?: 0
-------------------------------------+-------------------------------------
Changes (by vorner):
* owner: vorner => jinmei
* totalhours: 0 => 1.29
Comment:
Hello
Replying to [comment:10 jinmei]:
> avoid misleading build/test results, I've created a new branch
> trac2899-2, first merging trac2823-regression and then the original
> trac2899. Please refer to trac2899-2 (trac2899 has been removed from
> the public repo).
>
> I'll merge this branch once both this ticket and trac2823-regression
> are ready.
It seems OK now.
> At least these shouldn't be the reason; boost::interprocess::file_lock
> doesn't require a binary library, and while managed_mapped_file didn't
> compile on sunstudio, I don't think file_lock had that problem. I
> actually don't know why we ended up the in-house version. I suspect
> it's a kind of NIH syndrome. In any case, it's true logging is the
> only user, and I believe it's safer to keep its subroutine unless we
> have a real strong reason for not using existing tools for general
> purpose of interprocess synchronization.
OK, then it may have been boost::thread or something. I know we had
trouble with some of them, but I'm not sure which one.
I think this can be merged.
--
Ticket URL: <https://bind10.isc.org/ticket/2899#comment:12>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development
More information about the bind10-tickets
mailing list