这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@Yonle
Copy link

@Yonle Yonle commented Aug 27, 2021

Change package name from com.termux.x11 as com.termux.wayland.

@ghost
Copy link

ghost commented Aug 27, 2021

nice work, will test it then merge it once everything goes well, although there maybe pending pull requests that needs to be finished

@Yonle
Copy link
Author

Yonle commented Aug 27, 2021

Me feel like speedrunner-man 😂😂😂

@ghost ghost added the enhancement New feature or request label Aug 27, 2021
@creepy-pasta101
Copy link
Contributor

Also.. @Yonle you need to modify termux-x11 package in X11-REPO

@Yonle

This comment has been minimized.

@RalfWerner
Copy link

RalfWerner commented Aug 27, 2021

@Yonle Please do it not! Hundreds of scripts have to be changed and what we do here is pure X11! I work for 3-4 years with T:X11
See also here in termux-x11 packages repo are only XClients!

@ghost
Copy link

ghost commented Aug 27, 2021

the com.termux.wayland seems to match with app name but in that case would require tests and changing the scripts as @RalfWerner says, the current package name seems reasonable for me enough

@Yonle
Copy link
Author

Yonle commented Aug 27, 2021

Uh oh. Well i had no idea.

I only open this PR since this extension creates wayland unix socket to termux only, Plus the package name itself also need to changed agter changing it's app name.

I only open this PR only if there are contributors who WANT to change the package name. And also if I see from your comments, to be honest. I really can't do anything. It depends on the choice of collaborators.

@Yonle
Copy link
Author

Yonle commented Aug 27, 2021

If collaborators or maintainer want to close this PR, Simply close it. It depends on your choice whether you want to or not, that's your business.

@Yonle
Copy link
Author

Yonle commented Aug 27, 2021

When i'm renaming the package name, I'm ONLY using both sed and for loop to rename this quickly which also mean all script content is also modified.

@RalfWerner
Copy link

RalfWerner commented Aug 27, 2021

@Yonle, @layaachmad did you have a Wayland Client who is not X? @Yonle try sed in OTS-script
There was a reason why Twaik (author of T:X11) called it so and not Wayland.

@ghost ghost added the question Further information is requested label Aug 27, 2021
@ghost
Copy link

ghost commented Aug 27, 2021

in case you have any reasons for package name change feel free to comment otherwise this PR will be closed

@ghost ghost removed enhancement New feature or request question Further information is requested labels Aug 27, 2021
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.

3 participants