Opened 12 years ago

Closed 10 years ago

#35863 closed update (fixed)

varnish update to 3.0.3

Reported by: robsonpeixoto@… Owned by: Gminfly
Priority: Normal Milestone:
Component: ports Version:
Keywords: haspatch Cc: ryandesign (Ryan Carsten Schmidt), nerdling (Jeremy Lavergne)
Port: varnish

Description

Varnish update to 3.0.3

Attachments (8)

3.0.3.patch (880 bytes) - added by robsonpeixoto@… 12 years ago.
Only upgrade and change the checksums
3.0.3-startupitem-reloadscript.patch (5.8 KB) - added by robsonpeixoto@… 12 years ago.
A lot of new things. A status script, a reload script and startupitem.
3.0.3-startupitem-reloadscript.patch​ (5.8 KB) - added by robsonpeixoto@… 12 years ago.
V2 - A lot of new things. A status script, a reload script and startupitem
startupitem_livecheck.patch (4.7 KB) - added by robsonpeixoto@… 12 years ago.
Add statupitem, livecheck and a reload vcl script
varnish-autoload-20121014.patch (1.3 KB) - added by robsonpeixoto@… 12 years ago.
A more simple solution.
20121116-flexibleStartupItem+codeStyle+VclReloadScript.patch (3.2 KB) - added by robsonpeixoto@… 12 years ago.
Now, a patch thats realy work well
20121231-varnish-startup.patch (5.2 KB) - added by robsonpeixoto@… 12 years ago.
Full Solution. Creating all files and directories that I forgotten on last patch.
20130104-varnish-startup.patch​ (4.1 KB) - added by robsonpeixoto@… 12 years ago.
With 4 spaces and without convert tab to space

Download all attachments as: .zip

Change History (16)

comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… removed
Owner: changed from macports-tickets@… to pmq@…

Please attach a patch that has functional changes or whitespace changes but not both.

Changed 12 years ago by robsonpeixoto@…

Attachment: 3.0.3.patch added

Only upgrade and change the checksums

Changed 12 years ago by robsonpeixoto@…

A lot of new things. A status script, a reload script and startupitem.

Changed 12 years ago by robsonpeixoto@…

V2 - A lot of new things. A status script, a reload script and startupitem

comment:2 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added

I updated varnish to 3.0.3 in r97982. I did not change the epoch line because there's no reason to do so when the version increases normally. The only reason to increase the epoch is if the version decreases.

What do we need to do with the other patches you've attached here? If you want something else committed, please attach a diff with only functional changes (not mixed with whitespace changes).

comment:3 in reply to:  2 Changed 12 years ago by robsonpeixoto@…

Replying to ryandesign@…:

I updated varnish to 3.0.3 in r97982. I did not change the epoch line because there's no reason to do so when the version increases normally. The only reason to increase the epoch is if the version decreases.

Thanks =D

What do we need to do with the other patches you've attached here? If you want something else committed, please attach a diff with only functional changes (not mixed with whitespace changes).

The patch 'startupitem_livecheck.patch' add startupitem, a reload script and livecheck.

Changed 12 years ago by robsonpeixoto@…

Attachment: startupitem_livecheck.patch added

Add statupitem, livecheck and a reload vcl script

Changed 12 years ago by robsonpeixoto@…

A more simple solution.

Changed 12 years ago by robsonpeixoto@…

Now, a patch thats realy work well

comment:4 Changed 12 years ago by robsonpeixoto@…

I'll create a patch with all need files.

Last edited 12 years ago by robsonpeixoto@… (previous) (diff)

Changed 12 years ago by robsonpeixoto@…

Full Solution. Creating all files and directories that I forgotten on last patch.

comment:5 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)

The problem with the last patch you attached is that it mixes whitespace and functional changes, and the whitespace appears to have indents at multiples of two spaces, but MacPorts' preferred whitespace style is to have indents at multiples of four spaces.

Could you attach a patch with functional changes only, that leaves the port's whitespace style as it is? We should leave whitespace changes to the discretion of the port maintainer for now.

Changed 12 years ago by robsonpeixoto@…

With 4 spaces and without convert tab to space

comment:6 Changed 12 years ago by robsonpeixoto@…

Done !

comment:7 Changed 10 years ago by mf2k (Frank Schima)

Owner: changed from pmq@… to normen@…
Version: 2.1.2

Assigning to new maintainer.

comment:8 Changed 10 years ago by nerdling (Jeremy Lavergne)

Cc: snc@… added
Resolution: fixed
Status: newclosed

Updated in r121137.

Note: See TracTickets for help on using tickets.