Merge lp:~elambert/drizzle/rm_my_symlink into lp:~drizzle-trunk/drizzle/7.2-staging

Proposed by Eric Lambert
Status: Merged
Merged at revision: not available
Proposed branch: lp:~elambert/drizzle/rm_my_symlink
Merge into: lp:~drizzle-trunk/drizzle/7.2-staging
Diff against target: None lines
To merge this branch: bzr merge lp:~elambert/drizzle/rm_my_symlink
Reviewer Review Type Date Requested Status
Drizzle Developers Pending
Review via email: mp+7659@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Eric Lambert (elambert) wrote :

fixed my fix, it turns out that simply casting the results of the symlink function to void did not remove the warning. Also, building on Fedora 10 revealed a warning that my Mac was not generating, have fixed that. Have confirmed this build on Mac OS X and Fedora 10

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'mysys/mf_format.cc'
--- mysys/mf_format.cc 2009-06-16 01:36:01 +0000
+++ mysys/mf_format.cc 2009-06-18 17:36:19 +0000
@@ -100,7 +100,7 @@
100 struct stat stat_buff;100 struct stat stat_buff;
101 char rp_buff[PATH_MAX];101 char rp_buff[PATH_MAX];
102 if ((!flag & MY_RESOLVE_SYMLINKS) || 102 if ((!flag & MY_RESOLVE_SYMLINKS) ||
103 (!lstat(to,&stat_buff)) && S_ISLNK(stat_buff.st_mode))103 (!lstat(to,&stat_buff) && S_ISLNK(stat_buff.st_mode)))
104 {104 {
105 if (!realpath(to,rp_buff))105 if (!realpath(to,rp_buff))
106 my_load_path(rp_buff, to, NULL);106 my_load_path(rp_buff, to, NULL);
107107
=== modified file 'plugin/archive/ha_archive.cc'
--- plugin/archive/ha_archive.cc 2009-06-16 22:00:52 +0000
+++ plugin/archive/ha_archive.cc 2009-06-18 17:36:19 +0000
@@ -578,7 +578,8 @@
578 }578 }
579579
580 if (linkname[0])580 if (linkname[0])
581 (void)symlink(name_buff, linkname);581 if(symlink(name_buff, linkname) != 0)
582 goto error2;
582 fn_format(name_buff, table_name, "", ".frm",583 fn_format(name_buff, table_name, "", ".frm",
583 MY_REPLACE_EXT | MY_UNPACK_FILENAME);584 MY_REPLACE_EXT | MY_UNPACK_FILENAME);
584585

Subscribers

People subscribed via source and target branches