-
Notifications
You must be signed in to change notification settings - Fork 0
/
AUTOTOOLS-TODO
127 lines (70 loc) · 3.69 KB
/
AUTOTOOLS-TODO
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
REGRESSIONS:
- Port ActiveSync backend to non-recursive Automake.
IMPROVEMENTS:
- Add a check for qt-mobility for QtContacts backend.
- Probably client test should be built only when unit tests or integration tests
are enabled.
- Look at the note at the bottom of configure.ac:
# Avoid hard-coding paths in backends. These names are chosen so
# that a backend can alternatively use its own top-level configure
# with PKG_CHECK_MODULES(SYNCEVOLUTION, "syncevolution") to set them.
# need absolute path, use pwd instead of relative $srcdir
SYNCEVOLUTION_CFLAGS=-I`cd $srcdir && pwd`/src
SYNCEVOLUTION_LIBS=`pwd`/src/syncevo/libsyncevolution.la
AC_SUBST(SYNCEVOLUTION_CFLAGS)
AC_SUBST(SYNCEVOLUTION_LIBS)
Backends does not have their own top-level configure scripts, so usage of
absolute path have to be checked. For now this is worked around
in generated backends.am. Also, for relative path not $(srcdir) should be used
but $(builddir).
- Review CLEANFILES, DISTCLEANFILES, MAINTAINERCLEANFILES and MOSTLYCLEANFILES.
That is - check which files should be assigned to which of CLEAN variables.
- Check why distcheck outputs:
==================
All 0 tests passed
==================
There should have been at least one test being run. The same behavior exists
in old build system.
This is because neither unit tests nor integration tests are enabled by
default. If we would want to enable them then we should add
--enable-unit-tests and --enable-integration-tests into
DISTCHECK_CONFIGURE_FLAGS in toplevel Makefile.am.
- Check why there are so many failed tests when running `make check' explicitly.
The same number of failures exists in old build system:
Run: 583 Failure total: 528 Failures: 206 Errors: 322
- Tidy up configure.ac and some .am files.
- Maybe write scripts generating configs_xml.am, templates.am and profiles.am.
QUESTIONS:
- None.
PERSISTENT:
- Hunt for common variables clobbering.
- Silence more verbose output if found.
NITPICKS:
- Check if SyncEvolutionXML.c should always be recreated, patches always checked
and test-client always relinked.
- Maybe do not create /share/man/man1 directory if we do not put there anything.
- If backends are static libraries then do not create /lib/syncevolution,
because nothing is put here.
- Change $(foo) to @foo@ for all variables substituted by configure script. This
might be useful when looking for actual value of variable appearing out of
nowhere in .am file. Maybe make also all configure variables UPPER_CASE
and all local Automake variables lower_case.
- Lower autoreconf's warnings level later. Namely - don't warn about some
portability issues. The warnings about obsolete stuff should rather remain.
For now only some glib macros are using obsolete features.
- Should stamp files be marked as intermediate or rather as secondary files?
- Check why 'copying selected object files to avoid basename conflicts...' is
displayed between linking src/dbus/server/libsyncevodbusserver.la
and compiling src/gdbus/src_gdbus_libgdbussyncevo_la-debug.lo. This is
probably harmless.
TRACKING:
- Handle nobase prefixes. Link to track:
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9289
- Explanation why make -jX used to fail:
It failed because usually Automake generates dependencies of a library/program
by looking at its _LIBADD/_LDADD variable. But this generation doesn't work
correctly when _LIBADD/_LDADD variable has AC_SUBSTed variable - automake then
just discards them and resulting _DEPENDENCIES variable does not contain them.
As a workaround _DEPENDENCIES variable have to be written explicitly. Link to
track:
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=9320