dnsname icon indicating copy to clipboard operation
dnsname copied to clipboard

Fix build and install instructions

Open ivandov opened this issue 2 years ago • 7 comments

Instructions did not supply the command to build the binary (make) and therefore the make install would fail because the bin directory did not exist.

ivandov avatar May 02 '22 15:05 ivandov

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ivandov To complete the pull request process, please assign rhatdan after the PR has been reviewed. You can assign the PR to them by writing /assign @rhatdan in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

openshift-ci[bot] avatar May 02 '22 15:05 openshift-ci[bot]

You need to sign your commit message.

git commit -a --amend -s git push --force

rhatdan avatar May 02 '22 16:05 rhatdan

Signed the commit, and also realized this is almost identical to PR #93 which address issue #91.

Not sure if this PR should be closed since #93 looks to be further into the review process?

ivandov avatar May 03 '22 14:05 ivandov

CI is failing, seems unrelated. @Luap99 Is this the same thing that hit Podman CI?

mheon avatar May 03 '22 14:05 mheon

It related but no the same problem. The go version is just to old to compile ginkgo.

Looks like the VM images are very old cc @cevich

Luap99 avatar May 03 '22 14:05 Luap99

Looks like the VM images are very old

Yes, we just got buildah and skopeo updated last week. I need to make a jira card for updating all the others (there's a lot of them).

cevich avatar May 03 '22 17:05 cevich

https://github.com/containers/dnsname/pull/95

cevich avatar May 03 '22 17:05 cevich

What's the status of this? Just encountered this issue myself.

apetryla avatar Nov 27 '22 13:11 apetryla

This repo (and dnsname) are deprecated. We've switched all development efforts over to the Netavark network stack.

@Luap99 @baude Should we consider archiving this repo?

mheon avatar Nov 28 '22 19:11 mheon

If so, please LMK. I need to know to disable all the CI stuffs.

cevich avatar Nov 30 '22 15:11 cevich

I think CI can be disabled. Not so sure about archiving, dnsname is still shipped in distros and we might need to provide CVE fixes in the future?

Luap99 avatar Nov 30 '22 15:11 Luap99

Oh right, well if there's any CVE change, we need to keep CI alive then.

cevich avatar Nov 30 '22 15:11 cevich