Summary: | Incomplete/broken link in dev manual | ||
---|---|---|---|
Product: | Documentation | Reporter: | Fernando (likewhoa) <email> |
Component: | Devmanual | Assignee: | Gentoo Devmanual Team <devmanual> |
Status: | RESOLVED FIXED | ||
Severity: | trivial | ||
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | https://devmanual.gentoo.org/ebuild-writing/ebuild-maintenance/index.html | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
devmanual broken link fix
updated patch that replaces previous one devmanual-fix-broken-link-539630-1.diff |
Description
Fernando (likewhoa)
2015-02-10 15:30:02 UTC
Created attachment 396092 [details, diff]
devmanual broken link fix
(In reply to Fernando (likewhoa) from comment #1) > Created attachment 396092 [details, diff] [details, diff] > devmanual broken link fix +select UNCONFIRMED and RESOLVED (RESOLVED is important), then Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED? (In reply to Ulrich Müller from comment #2) > (In reply to Fernando (likewhoa) from comment #1) > > Created attachment 396092 [details, diff] [details, diff] [details, diff] > > devmanual broken link fix > > +select UNCONFIRMED and RESOLVED (RESOLVED is important), then > > Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED? I asked myself the same question but since it wasn't originally mentioned i left them out. Feel free to add as you see fit. (In reply to Fernando (likewhoa) from comment #3) > (In reply to Ulrich Müller from comment #2) > > (In reply to Fernando (likewhoa) from comment #1) > > > Created attachment 396092 [details, diff] [details, diff] [details, diff] [details, diff] > > > devmanual broken link fix > > > > +select UNCONFIRMED and RESOLVED (RESOLVED is important), then > > > > Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED? > > I asked myself the same question but since it wasn't originally mentioned i > left them out. Feel free to add as you see fit. Yes I think we need to mention all possible fields. Please attach an updated patch and I will commit it for you. Created attachment 396126 [details, diff]
updated patch that replaces previous one
(In reply to Markos Chandras from comment #4) > (In reply to Fernando (likewhoa) from comment #3) > > (In reply to Ulrich Müller from comment #2) > > > (In reply to Fernando (likewhoa) from comment #1) > > > > Created attachment 396092 [details, diff] [details, diff] [details, diff] [details, diff] [details, diff] > > > > devmanual broken link fix > > > > > > +select UNCONFIRMED and RESOLVED (RESOLVED is important), then > > > > > > Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED? > > > > I asked myself the same question but since it wasn't originally mentioned i > > left them out. Feel free to add as you see fit. > > Yes I think we need to mention all possible fields. Please attach an updated > patch and I will commit it for you. Updated and also removed non-existing fields. Created attachment 396128 [details, diff]
devmanual-fix-broken-link-539630-1.diff
This patch might be better since it suggest to select all possible fields in the status options list.
That looks ok. So now please attach a proper git patch and I will commit it for you :) |