-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat: rewrite in nushell #52
base: master
Are you sure you want to change the base?
Conversation
|
Wait you said this was in a docker container? |
We determined the issue only happens in a docker container. |
TestingInstall nutextNutext is the nu plugin I made that enables us to bring gettext functionality to nu programs. To install it, first clone the repo: git clone https://github.com/Elsie19/nu_plugin_nutext
make build
sudo make install Installing rpk2rpk2 is the actual program. To install it, first clone the repo: git clone -b nushell https://github.com/rhino-linux/rhino-pkg
sudo make install And you should be all good for testing! Notes
Reverting/uninstallingUninstallingNutextTo uninstall nutext, go back into the cloned sudo make uninstall rpk2This is harder because it likely overwrote your bash rpk, and I haven't tested exactly, but reinstalling rpk should do the trick: pacstall -I rhino-pkg-git
sudo rm -rf /usr/share/rhino-pkg/ |
Other stuffWe will have to keep |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently you need to add the plugin in a nu shell before being able to run it. I will look more into it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few issues:
- Unexpected behaviors when not giving a number as an input.
- No
rpk help
command. It seems like you copy/pastedrpk help
output torpk -h
, which presents issues. Furthermore it seems like auto-generated help information displays beneath this - Using
apt-remove
instead ofapt autoremove
- Running
rpk
without arguments returns an error, when it should return the content of the help command.
What i mean by the first point:
- When running
rpk install discord
and I select no options, it will say "selecting discord from snap" and then... fail? - When running
rpk remove discord
and I press enter without putting a number in, it throws an error.
Please ensure that rpk2 has consistent functionality with the first rpk.
That cannot be turned off btw. |
There is no Lines 200 to 203 in 0b7fc65
That's not a command, unless you meant Line 284 in 0b7fc65
In which case those are the same in apt.
Ok. |
Took you 2 months to figure out Line 122 in 90edb60
Don't keep asking me to test the code if you haven't done bare minimum testing yourself. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Firstly there is no rpk help
command, running rhino-pkg
will also throw an error. Running either rhino-pkg
with no arguments OR rhino-pkg help
should provide useful output to help the user as is the default behavior of the current rhino-pkg
implementation.
Running rpk cleanup gives output in these weird boxes with no data contained inside of them:
Information actually does get formatted into a box on rpk remove
as well:
Even more urgent is the fact that there are seemingly no checks for whether Nala is installed on the system. I didn't find any skimming the code, nor does it check when testing. Fix this.
Greetings, I am the user behind the backup and recovery ideas, which you can find here: #57 . I would be glad to continue the project and transfer those functions to this rewrite. I’d also like to contribute to the project, so I would appreciate your assistance in setting it up on my PC. Additionally, creating an app store version of this project seems like a great idea to me. This way, we can develop and maintain Rhino Linux's app store, similar to other distributions out there—take openSUSE's YaST, for example, which I greatly admire. I can try to build a very basic beta version soon so we can start working on it. Eventually, we could offer this app store to the world under Rhino Linux's name. Best regards, devtracer |
We still need a way to add i18n.Done.