diff options
| author |   José Bollo <jose.bollo@iot.bzh> | 2017-05-30 17:19:11 +0200 |
|---|---|---|
| committer | José Bollo <jose.bollo@iot.bzh> | 2017-05-30 17:19:11 +0200 |
| commit | 81f8b62bb2813d1cce388f10f39435d14042147d (patch) | |
| tree | fac4ac5f9702ba4369aa695c2dc7628378a7aec8 | |
| parent | d23600fbaf5abf4d584d58ff2664fa041e51f259 (diff) | |
Fix lack of message when setting exec bit
Nothing was telling that the content couldn't be
set as executable. Note that nothing is made to check
that a content (executable or not) exist.
Also note that installing several executables for
one widget isn't possible now.
Change-Id: I10373fa27cd8d1d90c292e3529d8a5eec62bb8f0
Signed-off-by: José Bollo <jose.bollo@iot.bzh>
| -rw-r--r-- | src/wgtpkg-install.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/wgtpkg-install.c b/src/wgtpkg-install.c index 44d2171..e4c3bbc 100644 --- a/src/wgtpkg-install.c +++ b/src/wgtpkg-install.c @@ -268,13 +268,17 @@ static int install_icon(const struct wgt_desc *desc) static int install_exec_flag(const struct wgt_desc *desc) { - int i; + int i, rc; if (desc->content_type) { i = sizeof exec_type_strings / sizeof *exec_type_strings; while (i) { - if (!strcasecmp(desc->content_type, exec_type_strings[--i])) - return fchmodat(workdirfd, desc->content_src, 0755, 0); + if (!strcasecmp(desc->content_type, exec_type_strings[--i])) { + rc = fchmodat(workdirfd, desc->content_src, 0755, 0); + if (rc < 0) + ERROR("can't make executable the file %s", desc->content_src); + return rc; + } } } return 0; |