Re: CVS commit: pkgsrc/comms

[jnemeth%victoria.tc.ca@localhost (John Nemeth)]

On Sep 24, 10:41am, "Joerg Sonnenberger" wrote:
}
} Module Name:  pkgsrc
} Committed By: joerg
} Date:         Fri May  4 16:06:14 UTC 2012
} 
} Modified Files:
}       pkgsrc/comms/asterisk: distinfo
}       pkgsrc/comms/asterisk10: Makefile distinfo
}       pkgsrc/comms/asterisk16: Makefile distinfo
}       pkgsrc/comms/asterisk18: Makefile distinfo
} Added Files:
}       pkgsrc/comms/asterisk/patches: patch-include_asterisk_inline__api.h
}       pkgsrc/comms/asterisk10/patches: patch-include_asterisk_inline__api.h
}       pkgsrc/comms/asterisk16/patches: patch-include_asterisk_inline__api.h
}       pkgsrc/comms/asterisk18/patches: patch-include_asterisk_inline__api.h
} 
} Log Message:
} Don't override optimizer settings with absurd levels.
} Fix inline definitions to work with C99 compiler.

     What is the purpose of these changes and why are they needed?
Without a good explanation, I'm backing them out.  I especially don't
like the first.  If upstream wants a high optimisation level, there
needs to be a very good reason to override it.  That doesn't count as a
bug fix.

}-- End of excerpt from "Joerg Sonnenberger"