aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDylan Baker <dylan@pnwbakers.com>2020-10-13 11:02:41 -0700
committerDylan Baker <dylan@pnwbakers.com>2020-11-13 09:00:35 -0800
commit5c74cccbb906ab92a75ae31262634a3e8b3f9dfa (patch)
tree44989650d5fb6e46b323b4bc8d0bed29323ce5a9
parent505f824cf177752c01b6193ec83f12e51035e844 (diff)
downloadmeson-5c74cccbb906ab92a75ae31262634a3e8b3f9dfa.zip
meson-5c74cccbb906ab92a75ae31262634a3e8b3f9dfa.tar.gz
meson-5c74cccbb906ab92a75ae31262634a3e8b3f9dfa.tar.bz2
run_unittests: use textwrap.dedent
So that editors that can fold code (vim, vscode, etc) can correctly fold functions, instead of getting confused by code that doesn't follow the current indention. Also, it makes the code easier to read.
-rw-r--r--mesonbuild/build.py9
-rwxr-xr-xrun_unittests.py51
-rw-r--r--test cases/failing/76 link with shared module on osx/test.json2
3 files changed, 34 insertions, 28 deletions
diff --git a/mesonbuild/build.py b/mesonbuild/build.py
index 78292f2..36d4e19 100644
--- a/mesonbuild/build.py
+++ b/mesonbuild/build.py
@@ -1359,11 +1359,12 @@ You probably should put it in link_with instead.''')
for link_target in self.link_targets:
if isinstance(link_target, SharedModule):
if self.environment.machines[self.for_machine].is_darwin():
- raise MesonException('''target links against shared modules.
-This is not permitted on OSX''')
+ raise MesonException(
+ 'target links against shared modules. This is not permitted on OSX')
else:
- mlog.warning('''target links against shared modules. This is not
-recommended as it is not supported on some platforms''')
+ mlog.warning('target links against shared modules. This '
+ 'is not recommended as it is not supported on some '
+ 'platforms')
return
class Generator:
diff --git a/run_unittests.py b/run_unittests.py
index 97110bc..0e8d62c 100755
--- a/run_unittests.py
+++ b/run_unittests.py
@@ -2943,20 +2943,22 @@ class AllPlatformTests(BasePlatformTests):
# the source tree leads to all kinds of trouble.
with tempfile.TemporaryDirectory() as project_dir:
with open(os.path.join(project_dir, 'meson.build'), 'w') as ofile:
- ofile.write('''project('disttest', 'c', version : '1.4.3')
-e = executable('distexe', 'distexe.c')
-test('dist test', e)
-subproject('vcssub', required : false)
-subproject('tarballsub', required : false)
-''')
+ ofile.write(textwrap.dedent('''\
+ project('disttest', 'c', version : '1.4.3')
+ e = executable('distexe', 'distexe.c')
+ test('dist test', e)
+ subproject('vcssub', required : false)
+ subproject('tarballsub', required : false)
+ '''))
with open(os.path.join(project_dir, 'distexe.c'), 'w') as ofile:
- ofile.write('''#include<stdio.h>
+ ofile.write(textwrap.dedent('''\
+ #include<stdio.h>
-int main(int argc, char **argv) {
- printf("I am a distribution test.\\n");
- return 0;
-}
-''')
+ int main(int argc, char **argv) {
+ printf("I am a distribution test.\\n");
+ return 0;
+ }
+ '''))
xz_distfile = os.path.join(self.distdir, 'disttest-1.4.3.tar.xz')
xz_checksumfile = xz_distfile + '.sha256sum'
zip_distfile = os.path.join(self.distdir, 'disttest-1.4.3.zip')
@@ -3615,8 +3617,8 @@ int main(int argc, char **argv) {
"""
tdir = os.path.join(self.unit_test_dir, '30 shared_mod linking')
out = self.init(tdir)
- msg = ('''WARNING: target links against shared modules. This is not
-recommended as it is not supported on some platforms''')
+ msg = ('WARNING: target links against shared modules. This is not '
+ 'recommended as it is not supported on some platforms')
self.assertIn(msg, out)
def test_ndebug_if_release_disabled(self):
@@ -7265,16 +7267,18 @@ class LinuxlikeTests(BasePlatformTests):
testdir = os.path.join(self.unit_test_dir, '61 identity cross')
nativefile = tempfile.NamedTemporaryFile(mode='w')
- nativefile.write('''[binaries]
-c = ['{0}']
-'''.format(os.path.join(testdir, 'build_wrapper.py')))
+ nativefile.write(textwrap.dedent('''\
+ [binaries]
+ c = ['{0}']
+ '''.format(os.path.join(testdir, 'build_wrapper.py'))))
nativefile.flush()
self.meson_native_file = nativefile.name
crossfile = tempfile.NamedTemporaryFile(mode='w')
- crossfile.write('''[binaries]
-c = ['{0}']
-'''.format(os.path.join(testdir, 'host_wrapper.py')))
+ crossfile.write(textwrap.dedent('''\
+ [binaries]
+ c = ['{0}']
+ '''.format(os.path.join(testdir, 'host_wrapper.py'))))
crossfile.flush()
self.meson_cross_file = crossfile.name
@@ -7287,9 +7291,10 @@ c = ['{0}']
'CC_FOR_BUILD': '"' + os.path.join(testdir, 'build_wrapper.py') + '"',
}
crossfile = tempfile.NamedTemporaryFile(mode='w')
- crossfile.write('''[binaries]
-c = ['{0}']
-'''.format(os.path.join(testdir, 'host_wrapper.py')))
+ crossfile.write(textwrap.dedent('''\
+ [binaries]
+ c = ['{0}']
+ '''.format(os.path.join(testdir, 'host_wrapper.py'))))
crossfile.flush()
self.meson_cross_file = crossfile.name
# TODO should someday be explicit about build platform only here
diff --git a/test cases/failing/76 link with shared module on osx/test.json b/test cases/failing/76 link with shared module on osx/test.json
index 4e2856f..0a32674 100644
--- a/test cases/failing/76 link with shared module on osx/test.json
+++ b/test cases/failing/76 link with shared module on osx/test.json
@@ -1,7 +1,7 @@
{
"stdout": [
{
- "line": "test cases/failing/76 link with shared module on osx/meson.build:8:0: ERROR: target links against shared modules."
+ "line": "test cases/failing/76 link with shared module on osx/meson.build:8:0: ERROR: target links against shared modules. This is not permitted on OSX"
}
]
}