Message ID | e57b513283313125c982fccaf52f49f25f688489.1389702128.git.christian.braunersorensen@prevas.dk |
---|---|
State | Rejected |
Delegated to: | Esben Haabendal |
Headers | show |
<christian.braunersorensen@prevas.dk> writes: > From: Christian Sørensen <christian.braunersorensen@prevas.dk> > > Signed-off-by: Christian Sørensen <christian.braunersorensen@prevas.dk> > --- > conf/paths/sdk.conf | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/conf/paths/sdk.conf b/conf/paths/sdk.conf > index aa6791e..6fd10d0 100644 > --- a/conf/paths/sdk.conf > +++ b/conf/paths/sdk.conf > @@ -22,8 +22,8 @@ sdk_infodir = "${sdk_datadir}/info" > sdk_mandir = "${sdk_datadir}/man" > sdk_docdir = "${sdk_datadir}/doc" > # Architecture dependent paths > -sdk_bindir = "${sdk_base_prefix}/bin" > -sdk_sbindir = "${sdk_base_prefix}/bin" > +sdk_bindir = "${sdk_exec_prefix}/bin" > +sdk_sbindir = "${sdk_exec_prefix}/bin" > sdk_libexecdir = "${sdk_exec_prefix}/libexec" > sdk_libdir = "${sdk_exec_prefix}/lib" > sdk_includedir = "${sdk_exec_prefix}/include" Why? Do you have recipe(s) that require this? If so, they are likely not really respecting the bindir and sbindir variables, and are thus broken and should be fixed. The reason for not having both bin and usr/bin in sdk is to avoid requiring the user to add two paths to their $PATH. Which recipe(s) is this related to? And is this the the "fixesx a bug for mingw" commit you mentioned in the cover letter? /Esben
diff --git a/conf/paths/sdk.conf b/conf/paths/sdk.conf index aa6791e..6fd10d0 100644 --- a/conf/paths/sdk.conf +++ b/conf/paths/sdk.conf @@ -22,8 +22,8 @@ sdk_infodir = "${sdk_datadir}/info" sdk_mandir = "${sdk_datadir}/man" sdk_docdir = "${sdk_datadir}/doc" # Architecture dependent paths -sdk_bindir = "${sdk_base_prefix}/bin" -sdk_sbindir = "${sdk_base_prefix}/bin" +sdk_bindir = "${sdk_exec_prefix}/bin" +sdk_sbindir = "${sdk_exec_prefix}/bin" sdk_libexecdir = "${sdk_exec_prefix}/libexec" sdk_libdir = "${sdk_exec_prefix}/lib" sdk_includedir = "${sdk_exec_prefix}/include"