This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project wmaker-crm.git.

The branch, next has been updated
       via  b7a1528833cf1abbe33067570f4f0d4a09f80ebc (commit)
       via  2ea61626296de392b4f72468d294bc62bf05fc9c (commit)
      from  5a730d63b20e7380b10e11918ebebc9c01c9224e (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://repo.or.cz/wmaker-crm.git/commit/b7a1528833cf1abbe33067570f4f0d4a09f80ebc

commit b7a1528833cf1abbe33067570f4f0d4a09f80ebc
Author: Alexandru Lazar <ala...@startmail.com>
Date:   Wed Apr 20 21:21:33 2016 +0300

    getstyle: fix wcopy_file paths
    
    Some (presumably stale) calls to wcopy_file used what appears to
    be an incorrect destination which did not always exist. This patch
    forces assets to be copied under <themedir>/<asset>, rather than
    <themedir><absolute path of asset>. It works by first getting the
    "new" path (i.e. the one that will be inserted in the property
    list), which is relative to <themedir> (and appears to be always
    in the root directory, too); it then copies the file to that path.
    
    This *may* have been the original intended behaviour, as the one
    it replaces clutters the path and leaks configuration data. In
    addition, the style file seems to store only the file's name, not
    the path relative to <themedir>, even when the file is copied with
    its full hierarchy.
    
    Signed-off-by: Alexandru Lazar <ala...@startmail.com>

diff --git a/util/getstyle.c b/util/getstyle.c
index f0de4c4..c2c61c3 100644
--- a/util/getstyle.c
+++ b/util/getstyle.c
@@ -171,7 +171,7 @@ static void findCopyFile(const char *dir, const char *file)
                        (void)wrmdirhier(ThemePath);
                return;
        }
-       wcopy_file(dir, fullPath, fullPath);
+       wcopy_file(dir, fullPath, file);
        wfree(fullPath);
 }
 
@@ -234,9 +234,10 @@ static void makeThemePack(WMPropList * style, const char 
*themeName)
 
                                p = strrchr(WMGetFromPLString(file), '/');
                                if (p) {
-                                       wcopy_file(themeDir, 
WMGetFromPLString(file), WMGetFromPLString(file));
-
                                        newPath = wstrdup(p + 1);
+
+                                       wcopy_file(themeDir, 
WMGetFromPLString(file), newPath);
+
                                        WMDeleteFromPLArray(value, 1);
                                        WMInsertInPLArray(value, 1, 
WMCreatePLString(newPath));
                                        free(newPath);
@@ -253,9 +254,10 @@ static void makeThemePack(WMPropList * style, const char 
*themeName)
 
                                p = strrchr(WMGetFromPLString(file), '/');
                                if (p) {
-                                       wcopy_file(themeDir, 
WMGetFromPLString(file), WMGetFromPLString(file));
-
                                        newPath = wstrdup(p + 1);
+
+                                       wcopy_file(themeDir, 
WMGetFromPLString(file), newPath);
+
                                        WMDeleteFromPLArray(value, 1);
                                        WMInsertInPLArray(value, 1, 
WMCreatePLString(newPath));
                                        free(newPath);
@@ -267,9 +269,10 @@ static void makeThemePack(WMPropList * style, const char 
*themeName)
 
                                p = strrchr(WMGetFromPLString(file), '/');
                                if (p) {
-                                       wcopy_file(themeDir, 
WMGetFromPLString(file), WMGetFromPLString(file));
-
                                        newPath = wstrdup(p + 1);
+
+                                       wcopy_file(themeDir, 
WMGetFromPLString(file), newPath);
+
                                        WMDeleteFromPLArray(value, 2);
                                        WMInsertInPLArray(value, 2, 
WMCreatePLString(newPath));
                                        free(newPath);

http://repo.or.cz/wmaker-crm.git/commit/2ea61626296de392b4f72468d294bc62bf05fc9c

commit 2ea61626296de392b4f72468d294bc62bf05fc9c
Author: Doug Torrance <dtorra...@piedmont.edu>
Date:   Sun Apr 24 18:40:45 2016 -0400

    wmmenugen: Fix segfault; argv[argc] doesn't exist.

diff --git a/util/wmmenugen.c b/util/wmmenugen.c
index 5839189..47e89f7 100644
--- a/util/wmmenugen.c
+++ b/util/wmmenugen.c
@@ -106,7 +106,7 @@ int main(int argc, char **argv)
        parse_locale(NULL, &env_lang, &env_ctry, &env_enc, &env_mod);
        terminal = find_terminal_emulator();
 
-       for (i = 1; i <= argc; i++)
+       for (i = 1; i < argc; i++)
        {
                if (strncmp(argv[i], "-parser", 7) == 0 &&
                    (argv[i][7] == '=' ||

-----------------------------------------------------------------------

Summary of changes:
 util/getstyle.c  | 17 ++++++++++-------
 util/wmmenugen.c |  2 +-
 2 files changed, 11 insertions(+), 8 deletions(-)


repo.or.cz automatic notification. Contact project admin crma...@gmail.com
if you want to unsubscribe, or site admin ad...@repo.or.cz if you receive
no reply.
-- 
wmaker-crm.git ("The Window Maker window manager")


-- 
To unsubscribe, send mail to wmaker-dev-unsubscr...@lists.windowmaker.org.

Reply via email to