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

Pawn package definition #262

Merged
merged 2 commits into from
Dec 28, 2017
Merged

Pawn package definition #262

merged 2 commits into from
Dec 28, 2017

Conversation

Southclaws
Copy link
Contributor

Since sampctl is getting to the point where it's production-ready, I'm encouraging people to start using it while developing libraries as it should make the process of testing and developing a lot smoother. It's designed so you don't need a pawn.json/pawn.yaml ("Pawn-Package Definition" file) in every repo, but having it there really helps with dependency management! These changes are purely additive and do not change your code or scripting style in any way, you don't need to use sampctl yourself but I encourage you to give it a try!

  • added sampctl package files for version control, so a user could now simply run sampctl package install Misiur/YSI-Includes to start using this library in their gamemode which would automatically pull in Zeex/amx_assembly.

@Y-Less
Copy link
Member

Y-Less commented Dec 28, 2017

It would be very helpful if you could also do this for the Branch_JIT branch, probably more useful since it has far more dependencies. Thanks.

@Y-Less Y-Less merged commit e9cf7a1 into pawn-lang:YSI.tl Dec 28, 2017
@Southclaws Southclaws deleted the pawn-package-definition branch December 28, 2017 14:42
@Southclaws
Copy link
Contributor Author

Yes I've been meaning to check out the JIT work actually, I'll do that.

However, branches actually haven't crossed my mind while working on sampctl so there's currently no way to say "I want this specific branch of this dependency" yet so I'll get working on some unit tests for branches (and probably use the YSI JIT Branch as a testbed), thanks!

@Y-Less
Copy link
Member

Y-Less commented Dec 28, 2017

I more meant can you commit this file to that branch, not can you make a dependency on that branch. But I guess both are useful.

@Y-Less
Copy link
Member

Y-Less commented Dec 28, 2017

Also, does something depend on YSI currently work, since the default branch is not master? Or does the fact that it is set as default suffice?

@Southclaws
Copy link
Contributor Author

Yeah I've recently been going through all my old releases and "packaging" them to test the tool out and YSI all works fine, seems the default branch is picked up implicitly by the Git library I'm using.

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