On 03/10/2014 02:30 AM, Lukas Slebodnik wrote:
On (08/03/14 02:00), Dmitri Pal wrote:
On 03/08/2014 01:40 AM, Dmitri Pal wrote:
On 03/07/2014 12:00 PM, Jakub Hrozek wrote:
On Fri, Mar 07, 2014 at 05:49:44PM +0100, Jakub Hrozek wrote:
On Fri, Mar 07, 2014 at 10:19:11AM -0500, Dmitri Pal wrote:
On 03/07/2014 10:11 AM, Jan Engelhardt wrote:
On Friday 2014-03-07 15:15, Lukas Slebodnik wrote:
attached patch addresses ticket #2193
Jan,
Do you have any other comments to the patch from previous mail?
No further comments at this time. Please, just get it
included already :}
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
I am preparing a new patch for ini and it will add two new entry
points but I can't finish the work until your patch is pushed.
ACK from me. make, make check, make distcheck and make rpms work fine.
SSSD also seems to be working fine after I upgraded ding-libs
to the new
RPMs.
Pushed to ding-libs master.
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
This push broke my deb builds on RHEL6:
s/deb/dev
Also removing EXTRA_ prefix seems to fix the issue.
[dpal@dpal ding-libs (master)]$ autoreconf -if
libtoolize: putting auxiliary files in AC_CONFIG_AUX_DIR, `build'.
libtoolize: copying file `build/ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIR, `m4'.
libtoolize: copying file `m4/libtool.m4'
libtoolize: copying file `m4/ltoptions.m4'
libtoolize: copying file `m4/ltsugar.m4'
libtoolize: copying file `m4/ltversion.m4'
libtoolize: copying file `m4/lt~obsolete.m4'
Makefile.am:259: variable `EXTRA_libini_config_la_DEPENDENCIES' is
defined but no program or
Makefile.am:259: library has `EXTRA_libini_config_la' as canonical
name (possible typo)
autoreconf: automake failed with exit status: 1
The EXTRA_*_DEPENDENCIES variable is useful for cases where you merely want
to augment the automake-generated _DEPENDENCIES variable rather than
replacing it.
http://www.gnu.org/software/automake/manual/html_node/Program-and-Library-Variables.html
It is a syntax warning in automake< 1.11.3 and syntax warnings
are enabled by default.
http://lists.gnu.org/archive/html/automake-patches/2010-10/msg00008.html
Makefile.am:259: variable `EXTRA_libini_config_la_DEPENDENCIES' is
defined but no program or library has `EXTRA_libini_config_la'
as canonical name (possible typo)
autoreconf: automake failed with exit status: 1
It would be better don't treat warnings as errors in automake.
The only negative of this solution is that library libini_conf will not be
recompiled with older version of automake if files listed in
EXTRA_*_DEPENDENCIES are changed. It is better to see warnings than override
automatically generated dependencies calculated by automake with
syntax: *_DEPENDENCIES
Patch is attached.
LS
But you use EXTRA prefix only with libini_config and not with other
libraries generated by this package. What is the difference?
I would have understood if all libraries used EXTRA prefix but it is not
the case. Or it is a bug?
Package that I have is 1.11.1. There is no other version available.
What is the recommendation with this issue?
--
Thank you,
Dmitri Pal
Sr. Engineering Manager for IdM portfolio
Red Hat Inc.
-------------------------------
Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
