Merge lp:~stefanor/ibid/feed-notify-help-605584 into lp:~ibid-core/ibid/old-trunk-1.6

Proposed by Stefano Rivera
Status: Merged
Approved by: Stefano Rivera
Approved revision: 963
Merged at revision: 968
Proposed branch: lp:~stefanor/ibid/feed-notify-help-605584
Merge into: lp:~ibid-core/ibid/old-trunk-1.6
Diff against target: 21 lines (+2/-1)
2 files modified
AUTHORS (+1/-0)
ibid/plugins/feeds.py (+1/-1)
To merge this branch: bzr merge lp:~stefanor/ibid/feed-notify-help-605584
Reviewer Review Type Date Requested Status
Jonathan Hitchcock Approve
Max Rabkin Approve
Review via email: mp+36933@code.launchpad.net

Commit message

Include notifying users in feed help

To post a comment you must log in.
Revision history for this message
Max Rabkin (max-rabkin) wrote :

Shouldn't you add Kevin Woodland to the copyright notice, since this was his patch?

Revision history for this message
Stefano Rivera (stefanor) wrote :

I suppose one should for attribution, but I think patches of this size are not copyrightable material

Revision history for this message
Max Rabkin (max-rabkin) :
review: Approve
Revision history for this message
Jonathan Hitchcock (vhata) wrote :

I have serious doubts about how you decide what size a patch is 'copyrightable'. Up to your discretion though.

review: Approve
964. By Stefano Rivera

Attribution

Revision history for this message
Stefano Rivera (stefanor) wrote :

I'm pretty sure that adding a word is not copyrightable. Personally I only add my name to the header of an ibid file when I've done non-trivial modifications to it (and that was the rule I followed when I generated them). I add his name to the list of contributors, though.

Revision history for this message
Max Rabkin (max-rabkin) wrote :

> I have serious doubts about how you decide what size a patch is 'copyrightable'.  Up to your discretion though.

If he had not included the patch, we would have generated the exact
same patch. Perhaps that's meaningful?

--Max

Revision history for this message
Stefano Rivera (stefanor) wrote :

I wasn't sure what the rules are under South African law here (but AFAIK such trivial patches are usually considered uncopyrightable in at least US law), so I had a read...

Did you know that the copyright holder the exclusive right for:
"performing the computer program in public;"

Anyway, can't find anything that really sheds light on this. Something else to be aware of is that unless the submitter provides a licence with his patch, he hasn't necessarily agreed to release it under the terms of the BSD license. However, with such trivial patches, one can take it as read that he has, as he can't really sue for the infringement of adding one word.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'AUTHORS'
2--- AUTHORS 2010-03-19 12:24:38 +0000
3+++ AUTHORS 2010-09-30 21:37:42 +0000
4@@ -20,3 +20,4 @@
5 * Neil Blakey-Milner
6 * Marco Gallotta
7 * Adrianna Pińska
8+ * Kevin Woodland
9
10=== modified file 'ibid/plugins/feeds.py'
11--- ibid/plugins/feeds.py 2010-05-11 23:02:12 +0000
12+++ ibid/plugins/feeds.py 2010-09-30 21:37:42 +0000
13@@ -100,7 +100,7 @@
14 add feed <url> as <name>
15 remove <name> feed
16 list feeds
17- poll <name> feed notify <channel> on <source>
18+ poll <name> feed notify ( <channel> | <user> ) on <source>
19 stop polling <name> feed
20 """
21 features = ('feeds',)

Subscribers

People subscribed via source and target branches