Skip to content

setup: install lldb-3.6/lldb-3.6-dev on ubuntu - #396

Closed
indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:feature/lldb
Closed

indutny wants to merge 1 commit into
nodejs:masterfrom
indutny:feature/lldb

Conversation

@indutny

@indutny indutny commented Apr 22, 2016

Copy link
Copy Markdown
Member

Required for llnode smoke testing.

cc @nodejs/build

@indutny

indutny commented Apr 22, 2016

Copy link
Copy Markdown
Member Author

NOTE: I'm not sure if the lldb-3.6 is available on all of these platforms. Please re-check!

@jbergstroem

Copy link
Copy Markdown
Member

Does it specifically require 3.6 or is it the lowest req?

@indutny

indutny commented Apr 22, 2016

Copy link
Copy Markdown
Member Author

I don't think it is the lowest required version, but I know for sure that it works starting from it.

@indutny

indutny commented Apr 23, 2016

Copy link
Copy Markdown
Member Author

Additional note: while not absolutely necessary, it will make things easier if the version of lldb will be the same across the machines. I'll need to supply path to lldb when running llnode's build script, and this path depends on the version.

@MylesBorins

Copy link
Copy Markdown
Contributor

@indutny why is only ubuntu included? Smoke testing also runs on fedora, osx, and ppc

@indutny

indutny commented Apr 26, 2016 •

Copy link
Copy Markdown
Member Author

@thealphanerd I'm sure that this thing works on OS X and Ubuntu, and OS X doesn't need any additional packages. I may take a look at fedora, but don't have access to ppc.

@jbergstroem

Copy link
Copy Markdown
Member

@indutny that's kind of why I asked. Can only assume that mileage will vary between the different systems. I know FreeBSD only have 3.7 or 3.8.

@indutny

indutny commented May 2, 2016

Copy link
Copy Markdown
Member Author

So anything actionable on this?

@jbergstroem

Copy link
Copy Markdown
Member

@indutny yeah; likely finding out all packages on the platforms we support and adding them to the playbooks. I can look at this, likely end of week. Is something held up by this at the moment?

@maclover7

Copy link
Copy Markdown
Contributor

ping @indutny -- is this still needed? If yes, this will need to be updated to change ansible, not setup

@BridgeAR

Copy link
Copy Markdown
Member

This should likely be closed. There was no progress for more than 1.5 years now.

@mmarchini

Copy link
Copy Markdown
Contributor

Closing since this has been inactive for quite some time now and #1025 downloads lldb before testing llnode. Feel free to reopen if you believe this should still land though.

@mmarchini mmarchini closed this Jul 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants