Opened 11 years ago

Closed 11 years ago

#4679 closed enhancement (fixed)

[patch] Add "Source Code" field to copy right entries

Reported by: mmadia Owned by: bonefish
Priority: normal Milestone: R1/alpha2
Component: Applications/AboutSystem Version: R1/alpha1
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

This is an attempt to implement listing Source Code links, as per this mailing list suggestion

It reads in or more "SourceURL" fields and display as a comma-separated list. However the email address field like syntax is not implemented. e.g. for subversion:

SourceURL: HaikuPorts Log <http://ports.haiku-files.org/wiki/dev-util/subversion/1.6.2/1>

The incomplete patch & screen shot are attached.

Attachments (2)

AboutSystem-SourceURL.png (45.7 KB ) - added by mmadia 11 years ago.
sourceURL-aboutsystem.patch (5.0 KB ) - added by mmadia 11 years ago.

Download all attachments as: .zip

Change History (7)

by mmadia, 11 years ago

Attachment: AboutSystem-SourceURL.png added

by mmadia, 11 years ago

Attachment: sourceURL-aboutsystem.patch added

comment:1 by alexbl, 11 years ago

X-Has-Patch: 1

It'd be nice if there was a convention for bubbling bugs with patches up...

comment:2 by mmadia, 11 years ago

Summary: Add "Source Code" field to copy right entries[patch] Add "Source Code" field to copy right entries

Normally [patch] is added to the Summary. Since this is an incomplete patch, I had some reservations on using it.

comment:3 by mmadia, 11 years ago

As seen in the screen shot, the patch is missing the ability to embed the URL into the leading text. eg, "Download" and "Instructions".

in reply to:  3 comment:4 by bonefish, 11 years ago

Status: newassigned

Replying to mmadia:

As seen in the screen shot, the patch is missing the ability to embed the URL into the leading text. eg, "Download" and "Instructions".

Yep, that's why I've ignored the patch so far. I was kind of hoping for an solution for that, too. :-) Will implement that now.

comment:5 by bonefish, 11 years ago

Resolution: fixed
Status: assignedclosed

Applied with small modifications in hrev34725.

Note: See TracTickets for help on using tickets.