fixes termux/termux-app#2415

[Hax4us]

No description provided.

[agnostic-apollo]

Thanks for this, will look into testing and merging after I push my local changes to master.

[babaric-dev]

Will this be merged? This is a very very useful thing that we miss now.

[agnostic-apollo]

Not for next release. The changes in this pull (not branch) are broken.

[babaric-dev]

Then should I reimplement this with the up-to-date repo?

[agnostic-apollo]

Like I said, current branch issues against master are not the issue, but the initial pull request changes submitted are, even though it was a clean code. There is a pretty big logical one too, should be noticeable to others too, one million imaginary dollar to whoever figures it out first... ;)

There also other issues as well regarding how this should ideally be implemented. Redoing with up-to-data master won't fix them. Fixing them all would be time consuming, time which I currently don't have since other higher priority things are being worked on, likely no time for testing either, unless someone else fixes them all as per quality and security standards required.

[PiprTuff]

May I hope to get this PR merged into master this year?

[agnostic-apollo]

Possibly this year, but not for next release or next 1-2 months if I have to do it.

[PiprTuff]

Possibly this year, but not for next release or next 1-2 months if I have to do it.

Its been very long since I have been waiting for this issue to be solved. No idea what other maintainers are doing. 😓

[huangyxi]

Hope this nice SAF file manipulation feature will be merged before the next release.