Repository navigation
Conversation
|
Adding a script to sync these files and add prefixes sounds good to me. Some questions:
|
If I understand it correctly, ASM means Assembly, not only means context/asm or fiber/asm... Assuming we introduce more ASM dependencies in the future, is it reasonable to mix all ASM files together? Or is it a bit confusing if we mix sync scripts with a lot of context asm files?
I'm also okay with including everything for the sake of simplicity.
Overwriting files is also good, but it cannot handle file name changes. This situation seems to happen before. The original file may become a dead file and lie in there quietly. Do you think it's a problem worth considering? |
|
This mostly looks fine to me, EDIT: I guess the |
|
@trowski Thanks for your suggestion. Does it look better now? It will move the old dir to the temp automatically. |
Is this really meant for release managers, or for updating boost/context from time to time (by whomever)? |
|
@cmb69 It's meant for anyone wishing to update Boost.Context. My assumption was that would be release managers, but I'm probably mistaken. It can be anyone that feels compelled to do so. I intended to do this manually as these asm files are infrequently updated, but since @twose put in the work to write an automated script we could make it part of another process so updates are bundled sooner. |
The script also automatically changes the API name of boost/context to avoid conflicts with third-party libraries
|
After reading related discussions, I tried to take some time to update this PR. Some changes:
|
|
Related: #20438 |
The script also automatically changes the API name of boost/context to avoid conflicts with third-party libraries.
PHP may conflict with third-party libraries which are using boost/context because all of them exports fcontext APIs. I think all boost/context users should have their own API name by adding prefixes (also including Swoole and Swow).
So I add a PHP script to download boost/context asm files and modify API names automatically, by the way, some of the code is copied from gen_stub.php.
It should be noted that the script would not remove the old dir automatically, otherwise, we need to introduce more code and have some security issues. So before we update, we must remove the context dir manually, then run the script.