-
Notifications
You must be signed in to change notification settings - Fork 56
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
tidy mods #70
tidy mods #70
Conversation
Huh, wow -- thanks so much! |
Thanks ! Still does it however
|
Huh, it worked when I tried it before shooting the PR. I can reproduce it now on a mac. Gotta think about it. I don't have a clue from the top of my head. |
Works for me. Maybe your go cache needs to know about the latest commit? try @master on the caddy-dns package. |
Nope the same, also it downloads before doing its build I think so cannot be cache issue.
I assume the cache folder is here?
I renamed both folders and ran again, same thing happen. I'm confused now, so without master it works but also it downloads much more than needed now? Not working Working? Output from
Again I always used master and it also works when I take the older commit, I use a batch script where the info is saved in so I dont have to remember it all the time, and its still unedited from back in march 2023 btw. when you dont include master in xcaddy in the beginning does it only take the official release or is it still the latest/master commit? |
@mohammed90 are we still into this issue? |
Mohammed's linked PR has the correct explanation; also from bcmills in the Gopher slack:
We're reconsidering our branching approach because I don't love this (personally). |
Can you give me an example for how I would build the latest master as today, tried various method to add a prelease tag, never succeed. I use this atm. xcaddy build master --with github.com/caddy-dns/cloudflare --with github.com/caddyserver/transform-encoder --with github.com/WeidiDeng/caddy-cloudflare-ip --with github.com/porech/caddy-maxmind-geolocation |
Try this:
|
Hi, thanks for taking time, however gives another error now.
I also tried just to use your url github.com/mohammed90/cloudflare instead of github.com/caddy-dns/cloudflare
|
Sigh, sorry, my bad. This one works:
This time, I checked it myself my creating a custom build using the same command. |
Great it works, thanks! 👍 👍 |
Fixes #69