RE: Samba 3.0 Alpha 21 - TZ environment variable.

2003-01-11 Thread Andrew Bartlett
On Sun, 2003-01-12 at 18:45, [EMAIL PROTECTED] wrote:
> Maybe the suggestion I made some time ago which worked for Solaris may work
> for HP as well, if the Tridge replacement doesn't.

The problem was that Samba_3_0 had not caught up with Samba HEAD.  (and
therefore the function you had trouble with also caused different
trouble for HPUX).

Andrew Bartlett

-- 
Andrew Bartlett [EMAIL PROTECTED]
Manager, Authentication Subsystems, Samba Team  [EMAIL PROTECTED]
Student Network Administrator, Hawker College   [EMAIL PROTECTED]
http://samba.org http://build.samba.org http://hawkerc.net



signature.asc
Description: This is a digitally signed message part


RE: Samba 3.0 Alpha 21 - TZ environment variable.

2003-01-11 Thread Clive . Elsum
Maybe the suggestion I made some time ago which worked for Solaris may work
for HP as well, if the Tridge replacement doesn't.
The timegm replacement within replace.c (see earlier e-mails).

 time_t timegm(struct tm *t)
{
  time_t tl, tb;
  struct tm *tg;

  tl = mktime (t);
  if (tl == -1)
{
  t->tm_hour--;
  tl = mktime (t);
  if (tl == -1)
return -1; /* can't deal with output from strptime */
  tl += 3600;
}
  tg = gmtime (&tl);
  tg->tm_isdst = 0;
  tb = mktime (tg);
  if (tb == -1)
{
  tg->tm_hour--;
  tb = mktime (tg);
  if (tb == -1)
return -1; /* can't deal with output from gmtime */
  tb += 3600;
}
  return (tl - (tb - tl));
}


[EMAIL PROTECTED]



-Original Message-
From: P Ranjit Kumar [mailto:[EMAIL PROTECTED]]
Sent: Saturday, 11 January 2003 9:45 AM
To: Andrew Bartlett; [EMAIL PROTECTED]
Cc: [EMAIL PROTECTED]
Subject: RE: Samba 3.0 Alpha 21 - TZ environment variable.


That will be a cool solution. I will take a look at it.

- Ranjit

-Original Message-
From: Andrew Bartlett [mailto:[EMAIL PROTECTED]]
Sent: Friday, January 10, 2003 2:34 PM
To: [EMAIL PROTECTED]
Cc: P Ranjit Kumar; [EMAIL PROTECTED]
Subject: Re: Samba 3.0 Alpha 21 - TZ environment variable.


On Sat, 2003-01-11 at 09:14, [EMAIL PROTECTED] wrote:
> On Fri, Jan 10, 2003 at 02:04:12PM -0800, P Ranjit Kumar wrote:
> > Hi
> >
> > Samba 3.0 Alpha21 tries to emulate the timegm() call. When it does that,
it
> > sets the TZ="" before it invokes the mktime() call. However, on HP-UX,
> > setting TZ="" makes the timezone to be EST but what we want is GMT for
> > time_t computation. This results in Samba failing to a join a domain.
>
> Adding platform specific code (like #ifdef HPUX) isn't something we
> want to move towards. Does HPUX not have timegm ? If not, can you
> work out a feature test patch that fixes it for HPUX rather than
> a platform specific one ?

Tridge's timegm() replacement doesn't use TZ any more, and has been
merged into Samba 3.0, for the next alpha.

Andrew Bartlett

--
Andrew Bartlett [EMAIL PROTECTED]
Manager, Authentication Subsystems, Samba Team  [EMAIL PROTECTED]
Student Network Administrator, Hawker College   [EMAIL PROTECTED]
http://samba.org http://build.samba.org http://hawkerc.net



RE: Samba 3.0 Alpha 21 - TZ environment variable.

2003-01-10 Thread P Ranjit Kumar
That will be a cool solution. I will take a look at it.

- Ranjit

-Original Message-
From: Andrew Bartlett [mailto:[EMAIL PROTECTED]]
Sent: Friday, January 10, 2003 2:34 PM
To: [EMAIL PROTECTED]
Cc: P Ranjit Kumar; [EMAIL PROTECTED]
Subject: Re: Samba 3.0 Alpha 21 - TZ environment variable.


On Sat, 2003-01-11 at 09:14, [EMAIL PROTECTED] wrote:
> On Fri, Jan 10, 2003 at 02:04:12PM -0800, P Ranjit Kumar wrote:
> > Hi
> >
> > Samba 3.0 Alpha21 tries to emulate the timegm() call. When it does that,
it
> > sets the TZ="" before it invokes the mktime() call. However, on HP-UX,
> > setting TZ="" makes the timezone to be EST but what we want is GMT for
> > time_t computation. This results in Samba failing to a join a domain.
>
> Adding platform specific code (like #ifdef HPUX) isn't something we
> want to move towards. Does HPUX not have timegm ? If not, can you
> work out a feature test patch that fixes it for HPUX rather than
> a platform specific one ?

Tridge's timegm() replacement doesn't use TZ any more, and has been
merged into Samba 3.0, for the next alpha.

Andrew Bartlett

--
Andrew Bartlett [EMAIL PROTECTED]
Manager, Authentication Subsystems, Samba Team  [EMAIL PROTECTED]
Student Network Administrator, Hawker College   [EMAIL PROTECTED]
http://samba.org http://build.samba.org http://hawkerc.net




Re: Samba 3.0 Alpha 21 - TZ environment variable.

2003-01-10 Thread Andrew Bartlett
On Sat, 2003-01-11 at 09:14, [EMAIL PROTECTED] wrote:
> On Fri, Jan 10, 2003 at 02:04:12PM -0800, P Ranjit Kumar wrote:
> > Hi
> > 
> > Samba 3.0 Alpha21 tries to emulate the timegm() call. When it does that, it
> > sets the TZ="" before it invokes the mktime() call. However, on HP-UX,
> > setting TZ="" makes the timezone to be EST but what we want is GMT for
> > time_t computation. This results in Samba failing to a join a domain.
> 
> Adding platform specific code (like #ifdef HPUX) isn't something we
> want to move towards. Does HPUX not have timegm ? If not, can you
> work out a feature test patch that fixes it for HPUX rather than
> a platform specific one ?

Tridge's timegm() replacement doesn't use TZ any more, and has been
merged into Samba 3.0, for the next alpha.

Andrew Bartlett

-- 
Andrew Bartlett [EMAIL PROTECTED]
Manager, Authentication Subsystems, Samba Team  [EMAIL PROTECTED]
Student Network Administrator, Hawker College   [EMAIL PROTECTED]
http://samba.org http://build.samba.org http://hawkerc.net



signature.asc
Description: This is a digitally signed message part


Re: Samba 3.0 Alpha 21 - TZ environment variable.

2003-01-10 Thread jra
On Fri, Jan 10, 2003 at 02:04:12PM -0800, P Ranjit Kumar wrote:
> Hi
> 
> Samba 3.0 Alpha21 tries to emulate the timegm() call. When it does that, it
> sets the TZ="" before it invokes the mktime() call. However, on HP-UX,
> setting TZ="" makes the timezone to be EST but what we want is GMT for
> time_t computation. This results in Samba failing to a join a domain.

Adding platform specific code (like #ifdef HPUX) isn't something we
want to move towards. Does HPUX not have timegm ? If not, can you
work out a feature test patch that fixes it for HPUX rather than
a platform specific one ?

Thanks,

Jeremy.