[dia: 42/105] #19: Code cleanup: s/FIXME/TODO/.
- From: Zander <zbrown src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [dia: 42/105] #19: Code cleanup: s/FIXME/TODO/.
- Date: Mon, 28 Jan 2019 19:22:51 +0000 (UTC)
commit c89c08c0d95011e03bcae03f53f1dc1f387ef9a6
Author: Eduard Nicodei <eddnicodei gmail com>
Date: Tue Jan 1 18:07:03 2019 +0000
#19: Code cleanup: s/FIXME/TODO/.
- Also remove old TODO in top meson.build, those dependencies should
be implicitly required by meson.
- Also add separate dependency for libc
(currently '-lc' flag is used; added "TODO").
app/meson.build | 2 +-
app/pixmaps/meson.build | 2 +-
doc/en/meson.build | 2 +-
meson.build | 15 +++++----------
objects/standard/pixmaps/meson.build | 2 +-
plug-ins/meson.build | 2 +-
shapes/flowchart/meson.build | 2 +-
sheets/meson.build | 2 +-
8 files changed, 12 insertions(+), 17 deletions(-)
---
diff --git a/app/meson.build b/app/meson.build
index 1c2db04f..176b2abf 100644
--- a/app/meson.build
+++ b/app/meson.build
@@ -63,7 +63,7 @@ dia_sources = [
'main.c'
]
-#FIXME: this is workaround for openbsd where libraries are in /usr/lib/local
+#TODO: this is workaround for openbsd where libraries are in /usr/lib/local
pkg_config = find_program('pkg-config')
pkg_config_libdir = run_command(pkg_config, '--libs-only-L', 'libxml-2.0').stdout().strip()
diff --git a/app/pixmaps/meson.build b/app/pixmaps/meson.build
index 751577c4..38407aff 100644
--- a/app/pixmaps/meson.build
+++ b/app/pixmaps/meson.build
@@ -23,7 +23,7 @@ icons = [ 'connectable.png',
'cursor-magnify-minus.png',
'cursor-magnify-plus.png' ]
-#FIXME: this is duplicated from lib/pixmaps. Any way to remove this duplication?
+#TODO: this is duplicated from lib/pixmaps. Any way to remove this duplication?
icon_pairs = []
foreach icon_file : icons
icon_pairs += ['dia_' + icon_file.split('.')[0].underscorify() + '_icon', files(icon_file)]
diff --git a/doc/en/meson.build b/doc/en/meson.build
index a385e334..db91a9fd 100644
--- a/doc/en/meson.build
+++ b/doc/en/meson.build
@@ -92,7 +92,7 @@ if xsltproc.found()
install : true,
install_dir : join_paths(get_option('mandir'), 'man1')
)
- # XXX: this should fixed in meson: https://github.com/mesonbuild/meson/issues/1550
+ # TODO: this should fixed in meson: https://github.com/mesonbuild/meson/issues/1550
#install_data(dia_man)
html_dir = join_paths(helpdir, 'en')
diff --git a/meson.build b/meson.build
index 66537d42..2bd68dd1 100644
--- a/meson.build
+++ b/meson.build
@@ -6,6 +6,7 @@ project('dia',
default_options : []
)
+#TODO: use libc_dep defined below.
add_global_link_arguments('-lc', language: 'c')
cc = meson.get_compiler('c')
@@ -18,18 +19,12 @@ libxml_dep = dependency('libxml-2.0', version : '>= 2.6.27')
gmodule_dep = dependency('gmodule-2.0')
libzlib_dep = dependency('zlib')
#TODO: this is optional. Should it be a disabler?
-libcairo_dep = dependency('cairo', required : false)
-
-libm_dep = cc.find_library('m', required : false)
+libcairo_dep = dependency('cairo')
+libm_dep = cc.find_library('m')
+libc_dep = cc.find_library('c')
# Used in pixmap csource generation.
gdk_pixbuf_csource = find_program('gdk-pixbuf-csource')
-# TODO
-#AC_PROG_INSTALL
-#AC_PROG_LN_S
-#AC_PROG_MAKE_SET
-#AC_ISC_POSIX
-
datadir = join_paths(get_option('prefix'),
get_option('datadir'))
@@ -60,7 +55,7 @@ endforeach
configuration_inc = include_directories('.')
-# FIXME: I don't think this should be defined.
+# TODO: I don't think this should be defined.
# To fix we should remove #ifdef HAVE_CONFIG_H checks from all source files.
add_project_arguments('-DHAVE_CONFIG_H', language : 'c')
diff --git a/objects/standard/pixmaps/meson.build b/objects/standard/pixmaps/meson.build
index bdf01f24..55b65598 100644
--- a/objects/standard/pixmaps/meson.build
+++ b/objects/standard/pixmaps/meson.build
@@ -11,7 +11,7 @@ icons = [ 'arc.png',
'text.png',
'zigzagline.png' ]
-#FIXME: this is duplicated from lib/pixmaps. Any way to remove this duplication?
+#TODO: this is duplicated from lib/pixmaps. Any way to remove this duplication?
icon_pairs = []
foreach icon_file : icons
icon_pairs += [icon_file.split('.')[0].underscorify() + '_icon', files(icon_file)]
diff --git a/plug-ins/meson.build b/plug-ins/meson.build
index de45b6d4..57130f99 100644
--- a/plug-ins/meson.build
+++ b/plug-ins/meson.build
@@ -26,7 +26,7 @@ subdir('pstricks')
#subdir('stress')
subdir('svg')
-# FIXME: requires visio-types.h
+# TODO: requires visio-types.h
#subdir('vdx')
#subdir('wmf')
diff --git a/shapes/flowchart/meson.build b/shapes/flowchart/meson.build
index e58a99fa..891da4fb 100644
--- a/shapes/flowchart/meson.build
+++ b/shapes/flowchart/meson.build
@@ -51,5 +51,5 @@ shape_sources = files(
'datasource.png'
)
-#FIXME: the folder should be capitalised as well.
+#TODO: the folder should be capitalised as well.
shapes += [[shape_sources, 'Flowchart']]
diff --git a/sheets/meson.build b/sheets/meson.build
index ddbb7103..c322cd31 100644
--- a/sheets/meson.build
+++ b/sheets/meson.build
@@ -97,7 +97,7 @@ if false
endif
sheet_files_targets = []
-# FIXME: use gettext:
+# TODO: use gettext:
# https://github.com/mesonbuild/meson/issues/1565
# https://github.com/systemd/systemd/commit/c81217920effddc93fb780cf8f9eb699d6fe1319
# https://wiki.gnome.org/MigratingFromIntltoolToGettext
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]