Attention is currently required from: fixeria, laforge.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    I am saying that for libosmo-abis, bcmtoolbox is not a direct (but indirect or implicit) dependency, pulled by ortp.

    If it's in the public header, imho it's not an indirect or implicit dependency, it's explicitly part of the ortp API. Otherwise it would be hidden in private headers or .c files.

    This is why I don't think it's a good idea to use API of bcmtoolbox.

    Agree.

    I am inclined to remove this talloc-to-ortp integration completely

    I don't have a strong opinion there, so as you wish.

To view, visit change 35502. To unsubscribe, or for help writing mail filters, visit settings.