Ubuntu
yelp package

Remove link to advanced-topics in Ubuntu table of contents

Bug #626352 reported by Matthew East
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
yelp (Ubuntu)
Fix Released
Low
Unassigned
Natty
Invalid
Low
Unassigned

Bug Description

Binary package hint: yelp

Further to a discussion in the docteam, we have decided to remove the "Advanced Topics" document from the ubuntu-docs package. As a result the link on the yelp frontpage to this document will break unless removed.

That link should therefore be removed. I'm attaching a debdiff.

Tags: patch
Revision history for this message
Matthew East (mdke) wrote :
Revision history for this message
Matthew East (mdke) wrote :

Sorry, having tried the debdiff and to build the package, the patch seems to break. I'm not sure why. Hopefully from the debdiff itself it's clear enough what needs to be changed and someone more familiar with yelp's patch system than me can sort this out.

Apologies for the confusion.

Revision history for this message
Matthew East (mdke) wrote :

Ok, I hope this debdiff works better. I've managed to build the package successfully.

Pedro Villavicencio (pedro)
Changed in yelp (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Matthew East (mdke) wrote :

This will lead to a broken link on the yelp frontpage for all users so I'm raising the importance and targetting.

Changed in yelp (Ubuntu):
importance: Low → Medium
milestone: none → ubuntu-10.10
Revision history for this message
Matthew East (mdke) wrote :

It seems that yelp isn't displaying the broken link now that the relevant document has been removed - it hides it. So I'm reducing the importance.

Changed in yelp (Ubuntu):
importance: Medium → Low
Kate Stewart (kate.stewart)
Changed in yelp (Ubuntu Natty):
milestone: ubuntu-10.10 → none
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

@Matthew:

Is this patch still needed? If the link is hidden, why would we still need to add a patch?

Thanks.

Changed in yelp (Ubuntu Natty):
status: Triaged → Incomplete
Revision history for this message
Bryce Harrington (bryce) wrote :

[Unsubscribing sponsors since looks like maybe patch isn't needed after all, c.f. previous comment. Please re-subscribe ubuntu-sponsors if/when it's ready to go.]

Changed in yelp (Ubuntu Natty):
status: Incomplete → New
status: New → Incomplete
Brian Murray (brian-murray)
tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for yelp (Ubuntu Natty) because there has been no activity for 60 days.]

Changed in yelp (Ubuntu Natty):
status: Incomplete → Expired
Matthew East (mdke)
Changed in yelp (Ubuntu Natty):
status: Expired → Invalid
Changed in yelp (Ubuntu):
status: Expired → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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