Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

user/terraform: new package #3291

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Fantome137
Copy link

Adds terrafrom, a tool for building and updating infrastructure as code idempotently

@q66
Copy link
Member

q66 commented Dec 16, 2024

there is already opentofu

@Fantome137
Copy link
Author

I teach a class that uses Terraform. It would be great to use Chimera Linux in it. I had not heard of OpenTofu and will check it out. For the class, I am going to have to stick to Terraform.

@q66
Copy link
Member

q66 commented Dec 17, 2024

they are the same thing, one is a fork due to terraform becoming proprietary software

@Fantome137
Copy link
Author

I totally understand, and I prefer the MPL, but the OpenTofu FAQ says explicitly that the projects will diverge after 1.6, they have both had releases since then, and they are not on the same version number now.

The course curriculum is explicitly Terraform, Kubernetes, AWS, Docker, and Jenkins. I cannot change any of that (so I cannot use Podman either). The Linux distro is not set though. The alternative libc and userland in Chimera Linux will be perfect when explaining containers.

Perhaps I will have to maintain my own repo


def post_install(self):
self.install_license("LICENSE")

Copy link
Member

Choose a reason for hiding this comment

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

spurious newline

url = "https://github.com/hashicorp/terraform"
source = f"{url}/archive/v{pkgver}.tar.gz"
sha256 = "3be0296246a96acb5d45cb354ca32417bffd55d8086a4e3fcd71073e4295bb17"
# Tests cause a segfault - compiled program works
Copy link
Member

Choose a reason for hiding this comment

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

this is not informative

also you can do this instead

def post_extract(self):
    # needs network/some of them segfault
    self.rm("internal/cloud/*_test.go", glob=True)

pkgrel = 0
build_style = "go"
makedepends = ["go"]
pkgdesc = "Tool for building and updating infrastructure as code idempotently"
Copy link
Member

Choose a reason for hiding this comment

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

just something like Tool for infrastructure as code should suffice

@Fantome137 Fantome137 force-pushed the terraform branch 2 times, most recently from d89329a to bff37c8 Compare December 19, 2024 05:31
@Fantome137
Copy link
Author

Changes applied

make_build_args = [
f"-ldflags=-X github.com/hashicorp/terraform/version.dev=no",
]
makedepends = ["go"]
Copy link
Member

Choose a reason for hiding this comment

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

this is hostmakedepends

sha256 = "3be0296246a96acb5d45cb354ca32417bffd55d8086a4e3fcd71073e4295bb17"

def post_extract(self):
self.rm("internal/cloud/*_test.go", glob=True)
Copy link
Member

Choose a reason for hiding this comment

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

add a comment why these are removed

@q66
Copy link
Member

q66 commented Dec 23, 2024

format your template with black

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.

2 participants