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)
Change History (16)
comment:1 Changed 12 years ago by ryandesign (Ryan Carsten Schmidt)
| Cc: | ryandesign@… removed |
|---|---|
| Owner: | changed from macports-tickets@… to pmq@… |
Changed 12 years ago by robsonpeixoto@…
| Attachment: | 3.0.3.patch added |
|---|
Only upgrade and change the checksums
Changed 12 years ago by robsonpeixoto@…
| Attachment: | 3.0.3-startupitem-reloadscript.patch added |
|---|
A lot of new things. A status script, a reload script and startupitem.
Changed 12 years ago by robsonpeixoto@…
| Attachment: | 3.0.3-startupitem-reloadscript.patch added |
|---|
V2 - A lot of new things. A status script, a reload script and startupitem
comment:2 follow-up: 3 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 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@…
| Attachment: | varnish-autoload-20121014.patch added |
|---|
A more simple solution.
Changed 12 years ago by robsonpeixoto@…
| Attachment: | 20121116-flexibleStartupItem+codeStyle+VclReloadScript.patch added |
|---|
Now, a patch thats realy work well
Changed 12 years ago by robsonpeixoto@…
| Attachment: | 20121231-varnish-startup.patch added |
|---|
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@…
| Attachment: | 20130104-varnish-startup.patch added |
|---|
With 4 spaces and without convert tab to space
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: | new → closed |
Updated in r121137.
Please attach a patch that has functional changes or whitespace changes but not both.