regexp.php file should be shipped with composer package about uap-php

[JustBlackBird]

I see several options:

• use git submodules
• move regexp file to a separated package and add it to composer dependencies.

[lucky7id]

👍 We have a custom bin location, and had to fork and modify the repo to make the download script to work.

[JustBlackBird]

@lucky7id I had a problem with CLI script too but it fixed now (see #7).

[lstrojny]

The option I would see is to add a sync job to the command line tool that syncs both test resources and regex.json and also creates the php file and commit all three of them. The benefit is that the package works out of the box, a fact I very much like. What I dislike is the fact that we have to actively release a new version once uap-core is updated. Opinions?

[lucky7id]

The composer scheme seems workable. Right now it's a very noticeable pain point among our team, having to update that every time.

[lstrojny]

The composer package variant would be even more hassle I suppose. I am thinking about having a cronjob that updates the regex.php nightly, runs the unit tests and automatically commits if the tests passed successfully. If not, manual intervention is needed.

[JustBlackBird]

@lstrojny don't you think that changes of PHP implementation and regexp should be separated?

[lstrojny]

@JustBlackBird it would be nicer indeed but we would still need to find a way to auto-update that package whenever uap-core changes + run the tests of uap-php. Given that I’m not sure it’s worth the trouble.

[JustBlackBird]

@lstrojny may be you are right. I'm only afraid of rapid increasing of the package's patch version.

[lstrojny]

@JustBlackBird let’s see how it goes. Closing the issue for now.

[JustBlackBird]

@lstrojny Thank you!