After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 711470 - Move tests from src/tests to test
Move tests from src/tests to test
Status: RESOLVED FIXED
Product: evolution-ews
Classification: Other
Component: Miscellaneous / EWS Core
3.11.x
Other Linux
: Normal normal
: ---
Assigned To: Evolution EWS maintainer(s)
Evolution EWS maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-11-05 10:24 UTC by Fabiano Fidêncio
Modified: 2013-11-05 23:12 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Update tar format to tar-ustar (1.52 KB, patch)
2013-11-05 10:25 UTC, Fabiano Fidêncio
committed Details | Review
tests: Move src/tests to tests and fix make distcheck (23.00 KB, patch)
2013-11-05 10:25 UTC, Fabiano Fidêncio
accepted-commit_now Details | Review
Bug #711470 - Move tests from src/tests to test and fix make distcheck (22.94 KB, patch)
2013-11-05 22:32 UTC, Fabiano Fidêncio
committed Details | Review

Description Fabiano Fidêncio 2013-11-05 10:24:01 UTC
See the attached patch
Comment 1 Fabiano Fidêncio 2013-11-05 10:25:48 UTC
Created attachment 258999 [details] [review]
Update tar format to tar-ustar

"tar-ustar selects the ustar format defined by POSIX 1003.1-1988. This format is believed to be old enough to be
portable. It fully supports empty directories. It can store file names with up to 256 characters, provided that the
file name can be split at directory separator in two parts, first of them being at most 155 bytes long. So, in most
cases the maximum file name length will be shorter than 256 characters. However you may run against broken tar
implementations that incorrectly handle file names longer than 99 characters (please report them to
bug-automake@gnu.org so we can document this accurately"

From http://www.gnu.org/software/automake/manual/html_node/List-of-Automake-options.html
Comment 2 Fabiano Fidêncio 2013-11-05 10:25:53 UTC
Created attachment 259000 [details] [review]
tests: Move src/tests to tests and fix make distcheck
Comment 3 Milan Crha 2013-11-05 18:10:55 UTC
Review of attachment 258999 [details] [review]:

I've nothing against, though make sure you can create a tarball and it'll be understood by other machine before committing this change. Evolution does't use this, but eds does, so it should be fine.
Comment 4 Milan Crha 2013-11-05 18:13:52 UTC
Review of attachment 259000 [details] [review]:

Feel free to commit, but make sure you can create a tarball, same as both 'make dist' and 'make distcheck' will work as expected, with or without disabled tests.

::: tests/Makefile.am
@@ +7,3 @@
+	-I$(top_builddir)/src \
+	-DTEST_FILE_DIR="\"$(top_srcdir)/tests/\"" \
+	-DG_LOG_DOMAIN=\"libeews\" \

use a different domain, like "libeews-tests"
Comment 5 Fabiano Fidêncio 2013-11-05 22:32:43 UTC
Created attachment 259049 [details] [review]
Bug #711470 - Move tests from src/tests to test and fix make distcheck
Comment 6 Fabiano Fidêncio 2013-11-05 23:12:52 UTC
Pushed to the master (3.11.2+)