Skip to content

Commit

Permalink
Removed cairo and gtk3 dependencies in VTR meta
Browse files Browse the repository at this point in the history 
Signed-off-by: Alessandro Comodi <[email protected]>
  • Loading branch information
@acomodi
acomodi committed Aug 14, 2019
1 parent b471e09 commit 249f715
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions vtr/meta.yaml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ requirements:
- {{ compiler('cxx') }}
host:
- bison
- cairo

This comment has been minimized.

Sign in to view

Copy link
@litghost

litghost Aug 15, 2019

Why did we remove these dependencies?
- cmake
- flex
- fontconfig
Expand All @@ -39,16 +38,13 @@ requirements:
- xorg-libxft
- tbb
- tbb-devel
- gtk3
run:
- readline <8
- tk
- cairo
- fontconfig
- xorg-libx11
- xorg-libxft
- tbb
- gtk3

#test:
# commands:
Expand Down

3 comments on commit 249f715

@acomodi
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@litghost they were not needed anymore with the latest fix on graphics from VTR upstream

@litghost
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@litghost they were not needed anymore with the latest fix on graphics from VTR upstream

But we were compiling with graphics in the VPR package all along? #8 was about making a no-gui version, but conda VPR should have the gui.

@acomodi
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I think that only gtk3 is needed now to enable graphics as it includes cairo as well.

Please sign in to comment.