Package logo's

Max Bolingbroke batterseapower at
Mon Oct 24 09:53:53 CEST 2011

On 24 October 2011 07:27, Bas van Dijk <v.dijk.bas at> wrote:
> I like the idea Max. Attached is the patch which implements it.
> There's only one issue: In doPackageRender I can't make the
> distinction between normal and candidate packages so I can't serve the
> correct logo url. So currently candidate logo's are broken. Do you
> have an idea to make them work again?

I see. It looks like doCandidateRender fixes up the result of
doPackageRender by modifying the rendPkgUri. What this suggests to me
is that we don't want to store the full logo URI but instead define a
rendLogoUri :: PackageRender -> Maybe String which constructs the URL
using the rendPkgUri.

I've modified your patch accordingly (attached) but haven't tested it.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: PackageLogos-v3.dpatch
Type: application/octet-stream
Size: 91740 bytes
Desc: not available
URL: <>

More information about the cabal-devel mailing list