please implement full diffs in the branches rss

Bug #256449 reported by TAC one
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned
loggerhead
Triaged
Wishlist
Unassigned
loggerhead-breezy
Triaged
Wishlist
Unassigned

Bug Description

Would be nice to have the option to subscribe an extended rss which could show the actuall diffs of every commit in the rss entries body

The diff should ideally feature code highlighting actually embedded in the html (inline style attributes, not a linked stylesheets as they would not be read by the most the feed readers)

Very long diffs (more than, say, 1000-2000 rows) could be truncated, but the whole thing would still be very useful for most projects.

I don't think you will ever implement this, but I really think it would be pretty cool :-)

Revision history for this message
TAC one (tacone) wrote :

Subscribing Martin Albisetti (loggerhead driver) for feedback. Hope you're interested Martin, if not, feel free to unsubscribe and sorry for the inconvenience.

Revision history for this message
Martin Albisetti (beuno) wrote :

This would be great to have.

Did you intend to have this in Launchpad's RSS, or Loggerhead's?

Revision history for this message
TAC one (tacone) wrote :

Hello, thanks for the feedback. I am currently using launchpad so I guess that would what I want. I just thought launchpad's branch rss were generated by loggerhead.

It doesn't really matters where the feature is implemented, as long as it's present.
Clicking on the feed icon in firefox and other browser would simply bring you the choice of a "bzr log" feed or a "bzr log + diff" feed. What do you think ?

Revision history for this message
Martin Albisetti (beuno) wrote :

It's interesting you would think that, because I think it doesn't make much sense to duplicate either. They are, btw, generated separately.

http://feeds.edge.launchpad.net/~loggerhead-team/loggerhead/trunk/branch.atom
http://bazaar.launchpad.net/~loggerhead-team/loggerhead/trunk/atom

I can add this capability to Loggerhead, but Launchpad ir out of my realm.
We should unify them anyway...

Revision history for this message
TAC one (tacone) wrote : Re: [Bug 256449] Re: please implement full diffs in the branches rss

> I can add this capability to Loggerhead,

That would be pretty cool.

> but Launchpad ir out of my realm.

Until it finally gets open-sourced it will be out of the realm of many :-)

> We should unify them anyway...

Actually the loggerhead one looks slightly better to me (it just lacks
bugs links). We'll see what the launchpad team decides to do about
that.

Martin Albisetti (beuno)
Changed in loggerhead:
importance: Undecided → Wishlist
status: New → Confirmed
Paul Hummer (rockstar)
Changed in launchpad-bazaar:
importance: Undecided → Wishlist
status: New → Triaged
Martin Albisetti (beuno)
Changed in loggerhead:
status: Confirmed → Triaged
Jelmer Vernooij (jelmer)
Changed in loggerhead-breezy:
status: New → Triaged
importance: Undecided → Wishlist
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.