Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.6.12
    • Fix Version/s: 1.8
    • Security Level: public
    • Labels:
      None
    • Environment:
      OpenWRT/LEDE cross compile environment
    • CPU Architecture:
      x86-64
    • Kernel:
      Linux
    • Userland:
      GNU/Linux
    • lsb_release:
      Don't have lsb_release around
    • Compiler:
      gcc
    • FreeSWITCH GIT Revision:
      FS-9836 #resolve [t38_gateway doesn't function properly after FS-9638 merge]
    • GIT Master Revision hash::
      6522572104a3addd9f59caefadc5f1109cb7170c
    • Target Version:
      1.8

      Description

      Hello all,

      FS doesn't compile on OpenWRT/LEDE due to -Werror and a few warnings (well, it does compile when I put ac_cv_gcc_supports_w_no_unused_result=no).

      The same warnings should be fixed in FS 1.6 branch. I'd appreciate it if you could put commit d28138ea4aa371185e719623f20ef6598f888c1f (FS-9812 fix label that is only used when zrtp or srtp are enabled) into the 1.6 branch as well (that'd fix all the warnings that are turned into errors when I compile 1.6).

      I've attached build logs that go hand in hand with the patches that I'll up as PRs.

      Kind regards,
      Sebastian

      1. mod-avmd-implicit-declaration.log
        870 kB
        Sebastian Kemper
      2. mod-sofia-redefine.log
        1.08 MB
        Sebastian Kemper
      3. mod-verto-implicit-declaration.log
        1.66 MB
        Sebastian Kemper
      4. sofia-implicit-declaration.log
        1.06 MB
        Sebastian Kemper

        Activity

        Hide
        Sebastian Sebastian Kemper added a comment -
        Hi all,

        I realize that my PR, although it is small, touches different parts of FreeSWITCH, all of which may have different maintainers. Should I instead send one PR per patch? It seems a bit over the top for a few one-line fixes, but if it helps then I have no issue with doing that.

        Kind regards,
        Sebastian
        Show
        Sebastian Sebastian Kemper added a comment - Hi all, I realize that my PR, although it is small, touches different parts of FreeSWITCH, all of which may have different maintainers. Should I instead send one PR per patch? It seems a bit over the top for a few one-line fixes, but if it helps then I have no issue with doing that. Kind regards, Sebastian

          People

          • Assignee:
            mikej Mike Jerris
            Reporter:
            Sebastian Sebastian Kemper
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development