Prechádzať zdrojové kódy

make paths relative to CWD, after all

the test suite actually relies on it. it would be possible to adjust it,
but there is not much reason to make paths relative to HOME (as we
support convenient tilde expansion). so use the least invasive approach,
which is simply the old behavior. adjust the documentation accordingly.

This reverts commit da5ce5d8f42a15a2842d6d14043452f1a7892a0a.
Oswald Buddenhagen 12 rokov pred
rodič
commit
15d57b95b7
3 zmenil súbory, kde vykonal 6 pridanie a 15 odobranie
  1. 1 1
      src/config.c
  2. 3 1
      src/mbsync.1
  3. 2 13
      src/util.c

+ 1 - 1
src/config.c

@@ -219,7 +219,7 @@ getopt_helper( conffile_t *cfile, int *cops, int ops[], char **sync_state )
 		while ((arg = get_arg( cfile, ARG_OPTIONAL, 0 )));
 		ops[M] |= XOP_HAVE_CREATE;
 	} else if (!strcasecmp( "SyncState", cfile->cmd ))
-		*sync_state = !strcmp( cfile->val, "*" ) ? nfstrdup( "*" ) : expand_strdup( cfile->val );
+		*sync_state = expand_strdup( cfile->val );
 	else
 		return 0;
 	return 1;

+ 3 - 1
src/mbsync.1

@@ -90,6 +90,8 @@ Lines starting with a hash mark (\fB#\fR) are comments and are ignored entirely.
 Configuration items are keywords followed by one or more arguments;
 arguments containing spaces must be enclosed in double quotes (\fB"\fR).
 All keywords (including those used as arguments) are case-insensitive.
+Bash-like home directory expansion using the tilde (\fB~\fR) is supported
+in all options which represent local paths.
 There are a few global options, the rest applies to particular sections.
 Sections are started by a section keyword and are terminated by an empty line
 or end of file.
@@ -177,7 +179,7 @@ Store's \fBTrash\fR. When using this, the remote Store does not need an own
 (Default: \fIno\fR)
 ..
 .SS Maildir Stores
-The reference point for relative \fBPath\fRs is $HOME.
+The reference point for relative \fBPath\fRs is the current working directory.
 .P
 As \fBmbsync\fR needs UIDs, but no standardized UID storage scheme exists for
 Maildir, \fBmbsync\fR supports two schemes, each with its pros and cons.

+ 2 - 13
src/util.c

@@ -337,35 +337,24 @@ expand_strdup( const char *s )
 	if (*s == '~') {
 		s++;
 		if (!*s) {
-		  rethome:
 			p = 0;
 			q = Home;
 		} else if (*s == '/') {
-			p = s + 1;
+			p = s;
 			q = Home;
 		} else {
 			if ((p = strchr( s, '/' ))) {
 				r = my_strndup( s, (int)(p - s) );
 				pw = getpwnam( r );
 				free( r );
-				p++;
 			} else
 				pw = getpwnam( s );
 			if (!pw)
 				return 0;
 			q = pw->pw_dir;
 		}
-		if (!p)
-			return nfstrdup( q );
-	  retjoin:
-		nfasprintf( &r, "%s/%s", q, p );
+		nfasprintf( &r, "%s%s", q, p ? p : "" );
 		return r;
-	} else if (*s != '/') {
-		if (*s == '.' && !s[1])
-			goto rethome;
-		p = s;
-		q = Home;
-		goto retjoin;
 	} else
 		return nfstrdup( s );
 }