# [INTERNALS-WIN] Re: [PHP-CVS] cvs: php-src /win32/build confutils.js

- Thread: https://externals.io/message/39184
- Original archive: https://news-web.php.net/php.internals/39184
- Started by Steph Fox on 2008-07-22T13:31:02+00:00
- Messages: 5
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #39184 — Steph Fox — 2008-07-22T13:31:02+00:00

URL: https://externals.io/message/39184#39184

> > […]
>
> There will be, because obviously that code is working in 5_3.

Sorry to harp on about this, but I still can't break it here. No idea why it 
should have failed in HEAD on the snaps box. The file was 100% identical to 
the 5_3 version until you committed your changes, and the 5_3 snaps aren't 
broken. And the point of reported breakage is even weirder - it comes on an 
equivalent line to AC_DEFINE().

Does anyone know the conditions that broke the script in HEAD, e.g. which 
extension/what was missing? Even throwing away the libxml libraries doesn't 
cause an error here.

Thanks,

- Steph

---

## #39187 — Pierre Joye — 2008-07-22T14:17:18+00:00

URL: https://externals.io/message/39184#39187 · In reply to: #39184

>
> > […]
>
> Sorry to harp on about this, but I still can't break it here. No idea why it
> should have failed in HEAD on the snaps box. The file was 100% identical to
> the 5_3 version until you committed your changes, and the 5_3 snaps aren't
> broken. And the point of reported breakage is even weirder - it comes on an
> equivalent line to AC_DEFINE().
>
> Does anyone know the conditions that broke the script in HEAD, e.g. which
> extension/what was missing? Even throwing away the libxml libraries doesn't
> cause an error here.

Easiest way to reproduce the runtime error in confutil is to empty the
lib/ dir and leave only resolv's files.

Cheers,
-- 
Pierre

http://blog.thepimp.net | http://www.libgd.org

---

## #39188 — Steph Fox — 2008-07-22T14:31:01+00:00

URL: https://externals.io/message/39184#39188 · In reply to: #39187

> > […]
>
> Easiest way to reproduce the runtime error in confutil is to empty the
> lib/ dir and leave only resolv's files.

Heh...

Checking for arpa\nameser.h ...  ..\bindlib_w32
Checking for library resolv_a.lib ... <not found>
Checking for library resolv.lib ... <not found>
ERROR: We really need that arpa\nameser.h file - it is part of the bindlib 
package

Actually with that fixed, the thing *still* works for me in HEAD. Sounds 
like a good idea but... doesn't help here.

I'm going to drop this for now and investigate when I've time. But it really 
wasn't a nasty merging accident (unlike the other one), it's just a weird 
and - for me at least - unreproducible error.

- Steph

---

## #39190 — Pierre Joye — 2008-07-22T14:35:37+00:00

URL: https://externals.io/message/39184#39190 · In reply to: #39188

>
> > […]
>
> Heh...
>
> Checking for arpa\nameser.h ...  ..\bindlib_w32
> Checking for library resolv_a.lib ... <not found>
> Checking for library resolv.lib ... <not found>
> ERROR: We really need that arpa\nameser.h file - it is part of the bindlib
> package
>
> Actually with that fixed, the thing *still* works for me in HEAD. Sounds
> like a good idea but... doesn't help here.
>
> I'm going to drop this for now and investigate when I've time. But it really
> wasn't a nasty merging accident (unlike the other one), it's just a weird
> and - for me at least - unreproducible error.

It was broken, it works now, problem solved, end of the discussion.
Waiting for your patches if you need something else in HEAD.


-- 
Pierre
http://blog.thepimp.net | http://www.libgd.org

---

## #39192 — Steph Fox — 2008-07-22T14:49:13+00:00

URL: https://externals.io/message/39184#39192 · In reply to: #39190

> It was broken, it works now, problem solved, end of the discussion.

Well not really, because HEAD and 5_3 are now different and the fact that we 
don't know *why* it failed in HEAD means we don't know if there's a problem 
in 5_3.

> Waiting for your patches if you need something else in HEAD.

Oh please. I missed ONE line in ONE merge. That is not a reason to play this 
game.

- Steph
