aerospike / php-client

Aerospike Client for PHP 8
https://aerospike.github.io/php-client/
9 stars 2 forks source link

Cosmetic fixes #5

Closed thias closed 9 months ago

thias commented 11 months ago
thias commented 10 months ago

Clearly not all of them, as the Makefile is still a mix of spaces and tabs and the README is pretty much the same, with extesnion, the super-obsolete PHP short open tag and all. Do you want me to quickly rebase all of these minor cosmetic enhancements?

vmsachin commented 9 months ago

sure, can you rebase the changes, ill merge them

thias commented 9 months ago

I've updated the README changes, keeping them minimal. I didn't touch the Makefile this time.

Would a more intrusive PR ripping out composer support and providing more generic build+install instructions be accepted? After seeing #12 it looks like most people seem to agree that there is no need to try and do too much, especially through fragile checks and potentially wrong assumptions, and that the extension's target audience knows what to do with a php module file.

PixiBixi commented 9 months ago

I would like to add something, Aerospike is not a common software used by anyone

When you need a soft like it, you have specific requirements and you perfectly know your infrastructure, there's no need to install it on the correct folder or whatever, just build it :)

vmsachin commented 9 months ago

@thias Sure, you can make a PR for the a better build process. Even we are looking into bettering the build for the php client. Any help in this area would be appreciated. Thanks!