Opened 7 years ago

Closed 5 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 Schmidt), nerdling (Jeremy L)
Port: varnish

Description

Varnish update to 3.0.3

Attachments (8)

3.0.3.patch (880 bytes) - added by robsonpeixoto@… 7 years ago.
Only upgrade and change the checksums
3.0.3-startupitem-reloadscript.patch (5.8 KB) - added by robsonpeixoto@… 7 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@… 7 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@… 7 years ago.
Add statupitem, livecheck and a reload vcl script
varnish-autoload-20121014.patch (1.3 KB) - added by robsonpeixoto@… 7 years ago.
A more simple solution.
20121116-flexibleStartupItem+codeStyle+VclReloadScript.patch (3.2 KB) - added by robsonpeixoto@… 7 years ago.
Now, a patch thats realy work well
20121231-varnish-startup.patch (5.2 KB) - added by robsonpeixoto@… 7 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@… 7 years ago.
With 4 spaces and without convert tab to space

Download all attachments as: .zip

Change History (16)

comment:1 Changed 7 years ago by ryandesign (Ryan 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 7 years ago by robsonpeixoto@…

Attachment: 3.0.3.patch added

Only upgrade and change the checksums

Changed 7 years ago by robsonpeixoto@…

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

Changed 7 years ago by robsonpeixoto@…

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

comment:2 Changed 7 years ago by ryandesign (Ryan 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 7 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 7 years ago by robsonpeixoto@…

Attachment: startupitem_livecheck.patch added

Add statupitem, livecheck and a reload vcl script

Changed 7 years ago by robsonpeixoto@…

A more simple solution.

Changed 7 years ago by robsonpeixoto@…

Now, a patch thats realy work well

comment:4 Changed 7 years ago by robsonpeixoto@…

I'll create a patch with all need files.

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

Changed 7 years ago by robsonpeixoto@…

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

comment:5 Changed 7 years ago by ryandesign (Ryan 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 7 years ago by robsonpeixoto@…

With 4 spaces and without convert tab to space

comment:6 Changed 7 years ago by robsonpeixoto@…

Done !

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

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

Assigning to new maintainer.

comment:8 Changed 5 years ago by nerdling (Jeremy L)

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

Updated in r121137.

Note: See TracTickets for help on using tickets.