IcaliaLabs / plis

Automated development tasks asked nicely
MIT License
40 stars 5 forks source link

AUR and local builds #17

Open Ziul opened 6 years ago

Ziul commented 6 years ago

What does this PR do?

After the PR be accepted, it's possible to create a version on AUR that use the git version, plus the release version

kurenn commented 6 years ago

You guys(@vovimayhem @thelastinuit) can you take a look at this, seems like a good contribution!

Thanks for it by the way @Ziul

thelastinuit commented 6 years ago

WOW!!! AUR!! it's been a long time I don't use it :( gotta go back!

thelastinuit commented 6 years ago

@Ziul The best practice in the go universe says that relative path are not the go way. Every package should have a unique import path and be imported everywhere using that same import path. HOWEVER, it does work. @vovimayhem what say yo?

Ziul commented 6 years ago

@thelastinuit That was something that really bothered me. The great motive for this PR was to include the plis-git on AUR, but I would need to build the package on local to the user. The way it was, I would make symlink from ~/ to some random path (probably /tmp), then go get to download plis, so then build it locally. It's madness the package to be self dependency. I asked two other AUR's maintainers what they think about it. Both reproved the way the package was doing. To be honest, that was my first program on Go that a touch, and i got surprised that ../ did the trick, but if there is a better way to do it avoiding the symlink, I will be OK with it.

thelastinuit commented 6 years ago

I think, take it with a grain of salt, there is a way with the latest updates but I'm not sure. Personally, I would accept the PR JUST to see it in AUR... it would be so badass. Let me get back to you @Ziul alright?

Ziul commented 6 years ago

Sure!

vovimayhem commented 6 years ago

Finally had some time to prioritize thing a bit around, and reviewing this PR

vovimayhem commented 6 years ago

...specially since #13 is still an [open and ongoing discussion](https://github.com/IcaliaLabs/plis/issues/13#issuecomment-3675509359.