Re: Current HEAD segfaults when built --with-tidy on Windows

php.internals

Andi Gutmans

22 years ago
I'll go and roll B3 this evening with or without this fix. I don't see this as a show stopper and there are other crash bugs anyway. Does anyone know if Edin is around? I need him to create the Win32 package. Is there anyone else who has the same system as his set up? Andi At 11:05 AM 12/20/2003 +0100, Sebastian Bergmann wrote:

Derick Rethans

22 years ago
On Sat, 20 Dec 2003, Andi Gutmans wrote:
> Does anyone know if Edin is around? I need him to create the Win32 package. > Is there anyone else who has the same system as his set up?
Edin makes the win32 package as he has all the libs etc. But it is weekend so IMO a bad time to roll a release. Why not wait until monday? I assume not many people checked your rc2 either yet. I now also now why you really need to switch back the version number as soon as you have tagged: (From a fresh CVS checkout): derick@kossu:/dat/dev/php/php-5.0dev$ ./buildconf You should not run buildconf in a release package. use buildconf --force to override this check. This is because we check if a version is "-dev", otherwise it's a release and users generally should not use ./buildconf in a release version. The same thing also broke generation of configure in the snapshots. regards, Derick

Andi Gutmans

22 years ago
At 03:55 PM 12/20/2003 +0100, Derick Rethans wrote:
>On Sat, 20 Dec 2003, Andi Gutmans wrote: > > > Does anyone know if Edin is around? I need him to create the Win32 package. > > Is there anyone else who has the same system as his set up? > >Edin makes the win32 package as he has all the libs etc. But it is >weekend so IMO a bad time to roll a release. Why not wait until monday? >I assume not many people checked your rc2 either yet.
Actually I prefer releasing it on the weekend because less ppl will be hurt if something goes wrong. Anyway, I'll put it up probably tomorrow in any case. We have often waited for the Win32 builds for a day or two.
>I now also now why you really need to switch back the version number as >soon as you have tagged: > >(From a fresh CVS checkout): >derick@kossu:/dat/dev/php/php-5.0dev$ ./buildconf >You should not run buildconf in a release package. >use buildconf --force to override this check. > >This is because we check if a version is "-dev", otherwise it's a >release and users generally should not use ./buildconf in a release >version. The same thing also broke generation of configure in the >snapshots.
That's not a good enough reason. It doesn't make sense to put such a restriction at a time when you're managing a release. In any case, maybe it's best to fix the snaps script to use --force :) Andi

Derick Rethans

22 years ago
On Sat, 20 Dec 2003, Andi Gutmans wrote:
> Actually I prefer releasing it on the weekend because less ppl will be hurt > if something goes wrong. Anyway, I'll put it up probably tomorrow in any > case. We have often waited for the Win32 builds for a day or two.
What can go wrong with a beta? ;-)
> That's not a good enough reason. It doesn't make sense to put such a > restriction at a time when you're managing a release. In any case, maybe > it's best to fix the snaps script to use --force :)
I fixed the snaps to do this... but I really don't understand why it is a problem to reset the version nr to -dev after tagging. We usually do this to prevent confusing people using a snapshot because they might think that they are using a release version (which the version string tells them). Derick

Andi Gutmans

22 years ago
At 07:51 PM 12/20/2003 +0100, Derick Rethans wrote:
>On Sat, 20 Dec 2003, Andi Gutmans wrote: > > > Actually I prefer releasing it on the weekend because less ppl will be hurt > > if something goes wrong. Anyway, I'll put it up probably tomorrow in any > > case. We have often waited for the Win32 builds for a day or two. > >What can go wrong with a beta? ;-)
True, true...
> > That's not a good enough reason. It doesn't make sense to put such a > > restriction at a time when you're managing a release. In any case, maybe > > it's best to fix the snaps script to use --force :) > >I fixed the snaps to do this... but I really don't understand why it is >a problem to reset the version nr to -dev after tagging. We usually do >this to prevent confusing people using a snapshot because they might >think that they are using a release version (which the version >string tells them).
Okay, I'll try and do that next time. I wasn't expecting such a long delay with the release which was mainly due to the --install-pear-packages crash. Andi