[dia: 91/105] #19: Minor TODO cleanups.



commit c7013ebe2c80368ab372a0dc5cee9c259ece79e9
Author: Eduard Nicodei <eddnicodei gmail com>
Date:   Wed Jan 23 21:10:56 2019 +0000

    #19: Minor TODO cleanups.

 plug-ins/meson.build      | 2 +-
 sheets/meson.build        | 6 +++---
 tests/exports/meson.build | 7 ++++---
 3 files changed, 8 insertions(+), 7 deletions(-)
---
diff --git a/plug-ins/meson.build b/plug-ins/meson.build
index c03c9891..f1e52684 100644
--- a/plug-ins/meson.build
+++ b/plug-ins/meson.build
@@ -26,7 +26,7 @@ subdir('python')  # Non-standard: shared_module
 subdir('stress')  # Non-standard: does not get installed.
 subdir('svg')
 subdir('vdx')
-#TODO: same here.
+# TODO: Port to Windows.
 if host_machine.system() != 'windows'
   subdir('wmf')  # Non-standard: Only installs if libemf present.
 endif
diff --git a/sheets/meson.build b/sheets/meson.build
index 6d2ce7f0..53b204b5 100644
--- a/sheets/meson.build
+++ b/sheets/meson.build
@@ -97,8 +97,8 @@ if false
 endif
 
 sheet_files_targets = []
-# TODO: use gettext:
-# https://github.com/mesonbuild/meson/issues/1565
+
+# TODO: use gettext: https://github.com/mesonbuild/meson/issues/1565
 # https://github.com/systemd/systemd/commit/c81217920effddc93fb780cf8f9eb699d6fe1319
 # https://wiki.gnome.org/MigratingFromIntltoolToGettext
 # https://www.gnu.org/software/gettext/manual/html_node/Preparing-ITS-Rules.html
@@ -118,7 +118,7 @@ checktrans_py = find_program('checktrans.py')
 
 custom_target(
     'sheet-translation-report',
-    build_by_default:  false,  # TODO: Currently broken anyway.
+    build_by_default:  false,  # TODO: tracked in issue #24.
     input: sheet_files_targets,
     output: 'sheet-translation-report',
     command: [checktrans_py, '@INPUT@'],
diff --git a/tests/exports/meson.build b/tests/exports/meson.build
index cbaf9d39..7fe30092 100644
--- a/tests/exports/meson.build
+++ b/tests/exports/meson.build
@@ -1,9 +1,10 @@
 test('exports',
     find_program('exports_test.sh'),
     args: [
-        # TODO: this is a bug in meson.  We don't need to
-        # specify full_path() https://github.com/mesonbuild/meson/issues/2681
-        diaapp.full_path()],
+        # TODO: We don't need to specify full_path().  See
+        # https://github.com/mesonbuild/meson/issues/2681
+        diaapp.full_path()
+    ],
     workdir: meson.current_source_dir(),
     env: run_env,
 )


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]