Opened 15 years ago
Closed 9 years ago
#20976 closed update (fixed)
viewcvs is now called viewvc
| Reported by: | ryandesign (Ryan Carsten Schmidt) | Owned by: | ryandesign (Ryan Carsten Schmidt) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | haspatch | Cc: | cooljeanius (Eric Gallager), jbenninghoff@…, kurthindenburg (Kurt Hindenburg) |
| Port: | viewcvs, viewvc |
Description
viewcvs changed names to viewvc some time ago since it now supports RCSes other than CVS. The viewcvs port should be copied to a new viewvc port and updated, and the old viewcvs port should state that viewvc replaces it, using the new replaced_by keyword.
Attachments (3)
Change History (12)
comment:1 Changed 15 years ago by nerdling (Jeremy Lavergne)
| Type: | update → request |
|---|
comment:2 Changed 10 years ago by cooljeanius (Eric Gallager)
| Cc: | egall@… added |
|---|
comment:3 Changed 10 years ago by jbenninghoff@…
New Portfile for viewvc attached, tested successfully on 10.9.4, and validated using port lint --nitpick. Patch for existing viewcvs Portfile attached, using replaced_by keyword and obsolete PortGroup.
Attached files:
viewvc-Portfile: new Portfile for viewvc
patch-viewvc-install.diff: patchfile, place in viewvc/files
patch-viewcvs-Portfile.diff: patch file for existing viewcvs Portfile
(remove existing viewcvs/files/patch-viewcvs-install)
Changed 10 years ago by jbenninghoff@…
| Attachment: | patch-viewvc-install.diff added |
|---|
patchfile, place in viewvc/files
Changed 10 years ago by jbenninghoff@…
| Attachment: | patch-viewcvs-Portfile.diff added |
|---|
patch file for existing viewcvs Portfile
comment:5 Changed 10 years ago by jmroot (Joshua Root)
| Keywords: | haspatch added |
|---|---|
| Type: | request → update |
| Version: | 1.8.0 |
comment:6 Changed 10 years ago by jmroot (Joshua Root)
One issue I noticed with the new portfile: I don’t think depends_build bin:python:python27 is correct, because python27 does not provide a ‘python’ executable, and I would think it is needed at runtime as well. There also doesn’t appear to be anything telling the build system which python to use.
comment:9 Changed 9 years ago by kurthindenburg (Kurt Hindenburg)
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
viewcvs obsoleted r144774
Cc Me!