Opened 11 years ago
Closed 11 years ago
#38288 closed defect (fixed)
rust @0.5: requires additional dependencies
| Reported by: | larryv (Lawrence Velázquez) | Owned by: | g5pw (Aljaž Srebrnič) |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | ports | Version: | |
| Keywords: | Cc: | cooljeanius (Eric Gallager) | |
| Port: | rust |
Description
Originally reported in #38279.
Rust apparently builds its own copy of LLVM or something, so it should have the corresponding dependencies. Ideally, we should be able to make it use an llvm-* port, but one step at a time.
It also seems to need build dependencies on Python and curl (although this one could be a bin dependency, since OS X comes with curl).
Change History (8)
comment:1 follow-up: 2 Changed 11 years ago by g5pw (Aljaž Srebrnič)
| Status: | new → assigned |
|---|
comment:2 Changed 11 years ago by larryv (Lawrence Velázquez)
Replying to g5pw@…:
As far as python goes, the usual MO is to add variants.
Just for a build-time choice? That doesn’t seem right to me. Unless something more than running some Python scripts is going on?
comment:3 follow-up: 4 Changed 11 years ago by g5pw (Aljaž Srebrnič)
It is possible to use MacPorts llvm. I think I'll set variants for each llvm verson. From a quick glance it seems like larryv is right, python is just a runtime dependency. System python may be ok. Unfortunately I'm unable to test a lot of system versions, so I'll fix issues as they come up.
comment:4 Changed 11 years ago by larryv (Lawrence Velázquez)
Replying to g5pw@…:
It is possible to use MacPorts llvm.
That’s quite an improvement, to say the least.
From a quick glance it seems like larryv is right, python is just a runtime dependency. System python may be ok. Unfortunately I'm unable to test a lot of system versions, so I'll fix issues as they come up.
If you don’t want to test a bunch of system versions (a reasonable decision, for sure), you might just want to declare a build dependency on bin:python2.7:python27 and make sure the build programs use python2.7. Off the top of my head, I don’t know which OS versions come with which Python versions.
comment:5 Changed 11 years ago by g5pw (Aljaž Srebrnič)
We use our LLVM now, see here. Still need to select the right python.
comment:6 Changed 11 years ago by cooljeanius (Eric Gallager)
curl won't be needed as a dependency any more once r103872 gets merged from larryv's sandbox into trunk
comment:7 Changed 11 years ago by larryv (Lawrence Velázquez)
As far as I can tell, Python is only required to build the bundled LLVM.
comment:8 Changed 11 years ago by larryv (Lawrence Velázquez)
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
curl is needed to fetch the older snapshot. So, if #38285 is resolved, this goes away. As far as python goes, the usual MO is to add variants.