[tools-issues] [Issue 83901] TMP does not have to be de fined

[cloph]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User cloph changed the following:

What|Old value |New value

  Status|RESOLVED  |VERIFIED





--- Additional comments from cl...@openoffice.org Mon Aug  3 16:13:50 + 
2009 ---
verified

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@tools.openoffice.org
For additional commands, e-mail: issues-h...@tools.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org

[tools-issues] [Issue 83901] TMP does not have to be de fined

[rene]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User rene changed the following:

What|Old value |New value

  Status|STARTED   |RESOLVED

  Resolution|  |FIXED





--- Additional comments from r...@openoffice.org Sat Jul 25 13:00:12 + 
2009 ---
fixed in cws tempdir.

only TMPDIR is used (as that#s the standard) and is honoured when already set in
the environemnt before ./configure/set_soenv

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@tools.openoffice.org
For additional commands, e-mail: issues-h...@tools.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org

[tools-issues] [Issue 83901] TMP does not have to be de fined

[vg]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User vg changed the following:

What|Old value |New value

Target milestone|OOo 3.1   |OOo Later





--- Additional comments from v...@openoffice.org Fri Mar 20 16:42:44 + 
2009 ---
.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@tools.openoffice.org
For additional commands, e-mail: issues-h...@tools.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org

[tools-issues] [Issue 83901] TMP does not have to be de fined

[er]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901





--- Additional comments from e...@openoffice.org Wed Mar 11 14:26:17 + 
2009 ---
Setting TMPDIR may be fine, but unsetting TMP even when it was set prior to
invoking configure is a NO-NO.
Plus the build breaks if PKGFORMAT=installed is used and extensions are to be
installed during that step. I don't know if TMPDIR will solve that.


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: issues-unsubscr...@tools.openoffice.org
For additional commands, e-mail: issues-h...@tools.openoffice.org


-
To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org
For additional commands, e-mail: allbugs-h...@openoffice.org

[tools-issues] [Issue 83901] TMP does not have to be de fined

[sb]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User sb changed the following:

What|Old value |New value

  CC|'cloph'   |'cloph,sb'





--- Additional comments from [EMAIL PROTECTED] Mon Nov 24 10:30:10 + 
2008 ---
I can only second kendy's <#desc7> request; please support SUSv3
()
TMPDIR behavior.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[mh]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User mh changed the following:

What|Old value |New value

Target milestone|---   |OOo 3.1





--- Additional comments from [EMAIL PROTECTED] Fri Nov 21 17:37:58 + 
2008 ---
please resolve this within 3.1 time frame, this looks not that difficult.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[kendy]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901





--- Additional comments from [EMAIL PROTECTED] Mon Nov 26 10:37:44 + 
2007 ---
> But one of $TMP or $TEMP should be defined, better when they are both
> defined.

vg: Let me disagree ;-)  The standard behavior in the unx-like systems is to 
use /tmp unless a variable TMPDIR is defined - then TMPDIR is used for the 
temporary files.  See http://en.wikipedia.org/wiki/TMPDIR - so please let's 
switch to the standard unx behavior: build.pl does not need any TMP or TEMP 
set to be able to run on unx system.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[vg]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901





--- Additional comments from [EMAIL PROTECTED] Mon Nov 26 10:23:16 + 
2007 ---
@cloph: I agree with you, it's not ok. I didn't expect that this variable in
normally missing in OOo environment. I committed temporary fix for that. But one
of $TMP or $TEMP should be defined, better when they are both defined. 
The TMP variable was guessed for years, and I do not think that guessing is the
best choice.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[rene]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901





--- Additional comments from [EMAIL PROTECTED] Sat Nov 24 20:10:29 + 
2007 ---
vg asked me some days when Issue 83414 would be fixed. For the record, here my
reply (I agreee that the TMP change is completely bogus):

--- snip ---
Date: Wed, 21 Nov 2007 20:33:16 +0100
From: Rene Engelhard <[EMAIL PROTECTED]>
To: Vladimir Glazounov <[EMAIL PROTECTED]>
Cc: [EMAIL PROTECTED], [EMAIL PROTECTED]
Subject: Re: issue 83414: how soon?
User-Agent: Mutt/1.5.17 (2007-11-01)

Hi,

Vladimir Glazounov wrote:
> while working on #81911# I intoduced a check for TMP variable in the build
> tool. Actually, it is right so, but because of missing fix for #83414# we

Wanna know the truth? I think this check is *COMPLETELY NONSENSE*. (And
AFAIS Rüdiger and Pavel do, too). (CCed)

Do you also patch any app out there to not set sane defaults for TMP but
require the user to set TMP and not working otherwise?
Or do you just override the envvar and the app will honour that you
want it something else?

The solution would be: restore the build tools old behaviour and let
people override it via envvar (yes, this works) when they want it
somwehere else?

> got problems with build bots - the build tool just refuses to work further
> without $TMP set.

Wheee. Well, you broke it. Even with 83414 "fixed" you still would have
the same problem as this issue ist just about "do not unset TMP/TEMP".
Not setting tmp to something. And even if that worked (see my try ...)
you *STILL* would not have TMP/TEMP set in the env. Plain Debian:

$ env | grep TMP
$ env | grep TEMP
$

What do you expect me to do? Hardcode TMP/TEMP to /tmp? What should I
harcode it for Windows then? Or Mac OS? Or OS/2?


> I see only quick-and-dirty solution for this problem now, that is restore
> the build tool old behaviour. But I would like to see the current
> implementation as soon as 83414 comes.
> Can you tell me your plans about the fix?

Yes. You should revert to something sane.

Regards,

Rene
--- snip --

vg: and your claim that configure unsets TMP is *wrong*. See comment #4 in Issue
83414. Even if we now changed this to TMP in the second line, the stuff will
still be not set because (see above) T(E)MP is not normally defined in a system.
And hardcoding it to /tmp or c:\temp or whatever is even more bogus.

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[cloph]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User cloph changed the following:

What|Old value |New value

  CC|''|'cloph'

 Assigned to|kendy |vg





--- Additional comments from [EMAIL PROTECTED] Sat Nov 24 18:23:18 + 
2007 ---
What is a pity is that such a change breaks building for all builders, breaks
building on tinderbox, etc and causes much problems that could have been avoided
if that change would have been announced beforehand.

And what is written in the last comment is a pity as well. If there has been an
apparently non-public exchange that discusses the need for that change, at least
the minutes or the results should be posted.

Why the heck should TMP be necessary when it build years without an explicitly
set variable? No matter what the answer is: An announcement would have helped to
avoid loads of problems and frustration.

And of course an announcement would have lead to a proper fix, instead of how it
was done. Configure must be the place where TMP should be checked for. (And it
is irrelevant that Sun doesn't use configure) Checking for TMP in build.pl
additionally to configure: OK, be it. Not checking for TMP in configure but
breaking because of it: Not OK. 

I hope I made my point clearly enough...

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[vg]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User vg changed the following:

What|Old value |New value

 Assigned to|vg|kendy





--- Additional comments from [EMAIL PROTECTED] Fri Nov 23 14:00:57 + 
2007 ---
@kendy: It's a pity, you wasn't in CC yesterday when we discussed exactly this
problem... "Even LinuxX86-64Env.Set.sh expicitely unsets it" - is not an
argument, it's a bug, that should be fixed. I will forward the mails...
Actually, something like your fix is already checked in as a temporary
solution... Important word is "temporary"...


-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[mh]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901


User mh changed the following:

What|Old value |New value

 Assigned to|mh|vg





--- Additional comments from [EMAIL PROTECTED] Fri Nov 23 13:44:12 + 
2007 ---
reassign

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

[tools-issues] [Issue 83901] TMP does not have to be de fined

[kendy]

To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=83901
 Issue #|83901
 Summary|TMP does not have to be defined
   Component|tools
 Version|680m237
Platform|All
 URL|
  OS/Version|All
  Status|NEW
   Status whiteboard|
Keywords|
  Resolution|
  Issue type|PATCH
Priority|P3
Subcomponent|solenv
 Assigned to|mh
 Reported by|kendy





--- Additional comments from [EMAIL PROTECTED] Fri Nov 23 13:35:44 + 
2007 ---
build.pl insists on TMP env. variable to be defined...  Why?  Even 
LinuxX86-64Env.Set.sh expicitely unsets it & breaks the build here ;-)  Patch 
to fix:

--- solenv/bin/build.pl
+++ solenv/bin/build.pl
@@ -2014,10 +2014,9 @@ sub get_tmp_dir {
 my $tmp_dir;
 if( defined($ENV{TMP}) ) {
 print_error("the \$TMP directory $ENV{TMP} does not exist! Please, 
create it first.") if (!-d $ENV{TMP});
-   $tmp_dir = $ENV{TMP} . '/';
+$tmp_dir = $ENV{TMP} . '/';
 } else {
-print_error("the \$TMP environment variable is not defined! Please, 
define it...");
-#   $tmp_dir = '/tmp/';
+$tmp_dir = '/tmp/';
 }
 $tmp_dir .= $$ while (-d $tmp_dir);
 $tmp_dir = CorrectPath($tmp_dir);

-
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]