<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
    <title></title>
  </head>
  <body text="#000000" bgcolor="#ffffff">
    Annoyingly it seems my mailer broke and my reply didn't send  ( or I
    don't see it yet because of still other mail problems ) so I
    apologise if you are getting this multiple times<br>
    <br>
    <div class="moz-text-flowed" style="font-family: -moz-fixed;
      font-size: 12px;" lang="x-unicode">Hi Petr
      <br>
      On 09/06/11 13:46, Petr Mladek wrote:
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">Noel Power
        píše v St 08. 06. 2011 v 17:36 +0100:
        <br>
      </blockquote>
      [...]
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">Hmm, I am not
        sure if we want to hardcode "/.config". If I understand it
        <br>
        correctly, there might be any path defined by XDG_CONFIG_HOME
        <br>
        environment variable. It could be even <i class="moz-txt-slash"><span
            class="moz-txt-tag">/</span>tmp/test<span
            class="moz-txt-tag">/</span></i>.blabla
        <br>
      </blockquote>
      yes, but here we are dealing with the case where XDG_CONFIG_HOME
      is not set. The patch mentioned in this thread sets the default
      config directory to $HOME/.config when XDG_CONFIG_HOME is not set
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">BTW: Is the
        XDG_CONFIG_HOME support a new feature?
        <br>
      </blockquote>
      no, not sure how long it is around but not new
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">  I thought
        that the
        <br>
        user configuration directory is defined
        <br>
        in /opt/libreoffice/program/bootstraprc. It is currently defined
        as:
        <br>
        <br>
              UserInstallation=$SYSUSERCONFIG/.libreoffice/3
        <br>
      </blockquote>
      afaik the XDG_CONFIG_HOME variable determines the value returned
      for the placeholder '$SYSUSERCONFIG'
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">I wonder if
        we could use $SYSUSERCONFIG to determine the default root
        <br>
        path where to serch for older configuration folders.
        <br>
        <br>
        If the XDG_CONFIG_HOME is an older feature. We should use
        <br>
        getenv('XDG_CONFIG_HOME') to find another root where to search
        for
        <br>
        older configurations.
        <br>
      </blockquote>
      not quite, but you are correct in that we should check if
      XDG_CONFIG_HOME is set, if it is then really things are as they
      were before and we don't imo need to do anything. Of course there
      is an argument that there is an existing bug where if you set
      XDG_CONFIG_HOME then also no userprofile migrations will happen.
      But I think that case is somewhat different because someone makes
      a conscious choice to change the config dir. In this case however
      ( because of the patch ) we have a situation where there is an
      enforced config directory location change.
      <br>
      <blockquote type="cite" style="color: rgb(0, 0, 0);">Best Regards,
        <br>
        Petr
        <br>
        <br>
        <br>
        PS: BTW: Why have you used
        <br>
        aUserInst = aUserInst.copy( 0, aUserInst.getLength() - sizeof(
        <br>
        XDG_CONFIG_PART ) + 2 );
        <br>
        <br>
        I would expect +1 instead of +2. But I am newer 100% sure about
        this
        <br>
        without testing <span class="moz-smiley-s3" title=";-)"><span>;-)</span></span>
        <br>
      </blockquote>
      yeah but the XDG_CONFIG_PART contains the leading '/' which I
      wanted to retain. But of course it's possible it isn't necessary
      and maybe it gets added anyway ( to be honest I didn't check and
      erred on the side of caution ). When I get a chance I'll check and
      change it ( and add the check for getenv ) ) Still interested in
      any other thoughts about the whole thing
      <br>
      <br>
      thanks
      <br>
      Noel<br>
    </div>
  </body>
</html>