-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
'libquery_parser.a', | ||
'liburl_formatter.a', | ||
], | ||
'linux': [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
have you checked these on linux and windows or are these a guess based on the mac libs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested on linux, windows and mac before submitting PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great, thanks!
I'll merge this after I finish updating to 53.0.2785.116 |
requires brave/libchromiumcontent#3 Auditors: @bridiver, @bbondy, @bsclifton
e116b6c
to
da7cc7d
Compare
Auditors: @bridiver, @bbondy, @bsclifton
da7cc7d
to
899d62c
Compare
++ just waiting for a small update to #4 and I'll merge both and get the builds started |
requires brave/libchromiumcontent#3 Auditors: @bridiver, @bbondy, @bsclifton
Auditors: @bridiver, @bbondy, @bsclifton