Opened 10 years ago

Closed 9 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:
Has a Patch: no 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 10 years ago.
sourceURL-aboutsystem.patch (5.0 KB) - added by mmadia 10 years ago.

Download all attachments as: .zip

Change History (7)

Changed 10 years ago by mmadia

Attachment: AboutSystem-SourceURL.png added

Changed 10 years ago by mmadia

Attachment: sourceURL-aboutsystem.patch added

comment:1 Changed 9 years ago by alexbl

X-Has-Patch: 1

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

comment:2 Changed 9 years ago by mmadia

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 Changed 9 years ago by 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".

comment:4 in reply to:  3 Changed 9 years ago by bonefish

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 Changed 9 years ago by bonefish

Resolution: fixed
Status: assignedclosed

Applied with small modifications in hrev34725.

Note: See TracTickets for help on using tickets.