Opened 9 years ago

Closed 9 years ago

#248 closed defect (fixed)

ASIO and std library dependency(?) issue

Reported by: jinmei Owned by: UnAssigned
Priority: high Milestone: 05. 3rd Incremental Release: Serious Secondary
Component: b10-auth Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity:
Sub-Project: Feature Depending on Ticket:
Estimated Difficulty: Add Hours to Ticket:
Total Hours: Internal?:

Description

I found ASIO didn't work on sol-10.lab. It didn't crash but the ModuleCCSession constructor blocked in an attempt of locking something in ASIO.

After many tries and errors I figured out we need r2185 (currently committed in branches/trac221). As commented, a better solution would be to generalize asio_link, but for now we need some workaround.

I plan to merge r2185 to trunk. It's trivial. Please someone do sanity check.

Proposed ChangeLog entry is:

  58.?	[bug]		jinmei
	Worked around an interaction issue between ASIO and standard C++
	library headers.  Without this ASIO didn't work: sometimes the
	application crashes, sometimes it blocked in the ASIO module.
	(Trac #XXX, svn rYYY)

I'm quite sure this is necessary, trivial, and safe. So I'll merge it anyway if it doesn't get reviewed quickly.

Subtickets

Change History (3)

comment:1 Changed 9 years ago by jinmei

  • Status changed from new to reviewing

comment:2 in reply to: ↑ description ; follow-up: Changed 9 years ago by each

I plan to merge r2185 to trunk. It's trivial. Please someone do sanity check.

Patch okay.

comment:3 in reply to: ↑ 2 Changed 9 years ago by jinmei

  • Resolution set to fixed
  • Status changed from reviewing to closed

Replying to each:

I plan to merge r2185 to trunk. It's trivial. Please someone do sanity check.

Patch okay.

Thanks, committed.

Note: See TracTickets for help on using tickets.