Mailing List Archive

Re: #1837: Error compiling VCL if probe is referenced before it is defined
#1837: Error compiling VCL if probe is referenced before it is defined
----------------------+--------------------
Reporter: fgsch | Owner:
Type: defect | Status: new
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: normal | Resolution:
Keywords: |
----------------------+--------------------

Comment (by fgsch):

FWIW, I'm not entirely sure this qualifies as a bug but calling a sub
before it's defined works just fine.

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1837#comment:2>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs
Re: #1837: Error compiling VCL if probe is referenced before it is defined [ In reply to ]
#1837: Error compiling VCL if probe is referenced before it is defined
----------------------+--------------------
Reporter: fgsch | Owner:
Type: defect | Status: new
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: normal | Resolution:
Keywords: |
----------------------+--------------------

Comment (by fgsch):

Patch including testcase attached.

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1837#comment:3>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs
Re: #1837: Error compiling VCL if probe is referenced before it is defined [ In reply to ]
#1837: Error compiling VCL if probe is referenced before it is defined
----------------------+---------------------------------------------
Reporter: fgsch | Owner: Federico G. Schwindt <fgsch@…>
Type: defect | Status: closed
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: normal | Resolution: fixed
Keywords: |
----------------------+---------------------------------------------
Changes (by Federico G. Schwindt <fgsch@…>):

* owner: => Federico G. Schwindt <fgsch@…>
* status: new => closed
* resolution: => fixed


Comment:

In [1ca122e4f9099e2d4b8d3d740783bced1151d78b]:
{{{
#!CommitTicketReference repository=""
revision="1ca122e4f9099e2d4b8d3d740783bced1151d78b"
Error earlier if we cannot find the probe

We don't have forward declarations in VCL. OK'd by phk@.
Fixes #1837.
}}}

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1837#comment:4>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs
Re: #1837: Error compiling VCL if probe is referenced before it is defined [ In reply to ]
#1837: Error compiling VCL if probe is referenced before it is defined
----------------------+---------------------------------------------
Reporter: fgsch | Owner: Federico G. Schwindt <fgsch@…>
Type: defect | Status: closed
Priority: normal | Milestone:
Component: varnishd | Version: trunk
Severity: normal | Resolution: fixed
Keywords: |
----------------------+---------------------------------------------

Comment (by Lasse Karstensen <lkarsten@…>):

In [3885d2c8fe3d5edca729b607cafc224349208c86]:
{{{
#!CommitTicketReference repository=""
revision="3885d2c8fe3d5edca729b607cafc224349208c86"
Error earlier if we cannot find the probe

We don't have forward declarations in VCL. OK'd by phk@.
Fixes #1837.
}}}

--
Ticket URL: <https://www.varnish-cache.org/trac/ticket/1837#comment:5>
Varnish <https://varnish-cache.org/>
The Varnish HTTP Accelerator

_______________________________________________
varnish-bugs mailing list
varnish-bugs@varnish-cache.org
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-bugs