use of dependency specification in pull breaks aptly graph #58

Closed
dupuy opened this Issue May 21, 2014 · 4 comments

Comments

Projects
None yet
2 participants
@dupuy

dupuy commented May 21, 2014

At least, this seems the most likely candidate for the source of this error, although I suppose the problem could be a too-complex mirror/repo/snapshot setup to fit on one page.

$ aptly graph
Loading mirrors...
Loading local repos...
Loading snapshots...
Loading published repos...
Generating graph...
Error: bad label format {Snapshot ubuntu-12-updates-main-20140512-0807-ff28|Pulled into 'ubuntu-12-updates-main-20140512-0807' with 'apt-cacher-ff28-20140430-0000' as source, pull request was: 'firefox(<<29.0) firefox-locale-de(<<29.0) firefox-locale-en(<<29.0)'|pkgs: 317}
dot: graph is too large for cairo-renderer bitmaps. Scaling by 0.502153 to fit
ERROR: exit status 1

@smira smira added the bug label May 21, 2014

@smira smira added this to the v0.6 milestone May 21, 2014

@smira

This comment has been minimized.

Show comment
Hide comment
@smira

smira May 21, 2014

Member

Probably the problem is incorrect escaping in aptly.

Member

smira commented May 21, 2014

Probably the problem is incorrect escaping in aptly.

@smira

This comment has been minimized.

Show comment
Hide comment
@smira

smira May 21, 2014

Member

Also #51 might help here.

Member

smira commented May 21, 2014

Also #51 might help here.

smira added a commit that referenced this issue May 28, 2014

Use gographviz escaping instead of homegrown escape function. #51 #58
It doesn't fix the bug #58, waiting for maintainer of gographviz to
accept the patch: https://code.google.com/p/gographviz/issues/detail?id=2
@smira

This comment has been minimized.

Show comment
Hide comment
@smira

This comment has been minimized.

Show comment
Hide comment
@smira

smira May 30, 2014

Member

Fixed in master.

Member

smira commented May 30, 2014

Fixed in master.

@smira smira closed this May 30, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment