Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

opkg support #62

Closed
urandom2 opened this issue Nov 15, 2015 · 11 comments · Fixed by #159, #161 or #162
Closed

opkg support #62

urandom2 opened this issue Nov 15, 2015 · 11 comments · Fixed by #159, #161 or #162

Comments

@urandom2
Copy link

It would be awesome if you could add support for opkg.

@icy
Copy link
Owner

icy commented Nov 16, 2015

Hi @arnottcr,

I will take a look .

@icy icy self-assigned this Nov 16, 2015
@icy
Copy link
Owner

icy commented Dec 2, 2015

Porting pacapt to support opkg is a real challenge. In OpenWRT embeded environment, (d)ash is used instead of bash; and other standard tools are implemented in a different fashion (Not related, but it's true that wget in OpenWRT doesn't support https.)

CC: @cmpitg and @Gnouc if you have any brilliant ideas.

@urandom2
Copy link
Author

urandom2 commented Dec 2, 2015

While I agree it would add complexity, you could have a vanilla POSIX header that determines the shell and then code paths for each major subgroup. I would imagine that this within the realm of what I am capable of, so if I get time I may submit a pull request, or be back with comments/questions.

Also, from what I have seen bash is great for interactivity, but not the fastest at executing POSIX scripts, so there could be merit in migrating to a pure POSIX implementation of pacapt.

(I agree it is odd that wget does not support tls, then again most package managers I have worked with, pacman included, prefer http and signatures over https.)

@cuonglm
Copy link
Collaborator

cuonglm commented Dec 2, 2015

@arnottcr pacapt code base relied heavily on non-POSIX shell features, example process substitution, local variable, globstar. Not all of them can be made to work in POSIX shell.

It need a big effort to porting it to a POSIX implementation.

@urandom2
Copy link
Author

urandom2 commented Dec 2, 2015

@Gnouc, thanks; I have not looked in depth at the pacapt codebase, but that seems quite reasonable. [sarcasm] I will submit a pull request when I have all the porting done. [/sarcasm]

@icy
Copy link
Owner

icy commented Dec 2, 2015

I think it's good to start new branch, e.g, paccapt_posix. I also need some testing support. (For now, all tests are done manually.) There are many things to do now :D

@icy
Copy link
Owner

icy commented Aug 2, 2018

@arnottcr @Gnouc

pacapt main part is now written in Dlang. WIP found from nd branch https://github.com/icy/pacapt/tree/nd.

I will take a look how the application can be built on OpenWRT. After that we can start new support for opkg.

@icy
Copy link
Owner

icy commented Jun 19, 2021

@arnottcr POSIX environment now can be supported thanks to #161 . Once it's merged I'll get back to this issue. Thanks.

@icy icy closed this as completed in #161 Jun 19, 2021
@icy icy reopened this Jun 19, 2021
@icy
Copy link
Owner

icy commented Jun 19, 2021

Due to wrong linking the ticket was closed automatically. I reopen it now. Thanks.

icy added a commit that referenced this issue Jun 19, 2021
@icy icy linked a pull request Jun 19, 2021 that will close this issue
4 tasks
@icy icy removed a link to a pull request Jun 20, 2021
4 tasks
@icy
Copy link
Owner

icy commented Jun 20, 2021

A quite complete opkg support : https://github.com/icy/pacapt/pull/162/files#diff-88f1c7341e39e99c75f394dc9cccbebdc080299d915d4839bfb85352740d9385

Please help to review PR and I'll release them next week. Thanks.

@icy icy closed this as completed in 23cdf4a Jun 21, 2021
icy added a commit that referenced this issue Jun 21, 2021
@icy icy linked a pull request Jun 21, 2021 that will close this issue
4 tasks
@icy
Copy link
Owner

icy commented Jun 21, 2021

I'm happy to release new version v3.0.0, its opkg support is ready :) I would have written the script with POSIX from the first day (kinda 11 years ago:D). This request is > 5 years ago, but it's still better to have something late than... never.

Enjoy opkg support: https://github.com/icy/pacapt/releases/tag/v3.0.0

edit: fix typ0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants