Skip to content
New issue

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

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

Already on GitHub? # to your account

Added: nala (and nala-legacy) #436

Merged
merged 2 commits into from
Sep 25, 2022

Conversation

philclifford
Copy link
Member

Closes #381
by implementing the solution given there

@philclifford philclifford changed the title Added nala (and nala-legacy) Added: nala (and nala-legacy) Aug 10, 2022
Copy link
Contributor

@natanjunges natanjunges left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please rebase with main and update to the new package metadata format.

@philclifford
Copy link
Member Author

Please rebase with main and update to the new package metadata format.

I'd love to but main is broken: (./deb-get prettylist fails so cog on the README fails)
prettylist is not tested in the current test btw - (and neither is cog functionality)

Here's what happened (this is on my "clean" upstream branch as a check after getting the same on my add-nala-squashed branch)

04:52 $ git pull -r upstream main 
From github.com:wimpysworld/deb-get
 * branch            main       -> FETCH_HEAD
Updating 39d40d8..687a3dc
Fast-forward
 CONTRIBUTING.md |  11 +++--
 README.md       |  12 ++++-
 deb-get         | 498 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------------
 debian/control  |   1 +
 debian/postinst |  20 +++++++++
 5 files changed, 372 insertions(+), 170 deletions(-)
 create mode 100755 debian/postinst
(base) ✔ ~/src/deb-get [official L|…7⚑ 11] 
04:52 $ ./deb-get prettylist
| Source   | Package Name   | Description   |
| :------: | :------------- | :------------ |
| [<img src=".github/debian.png" align="top" width="20" />](https://1password.com/) | `1password` | <i>The easiest way to store and use strong passwords.</i> |
| [<img src=".github/github.png" align="top" width="20" />](https://github.com/OrangeDrangon/android-messages-desktop/) | `android-messages-desktop` | <i>Run Android Messages as a desktop app.</i> |
  [!] ERROR! Missing required information of github package antimicrox:
URL=
VERSION_PUBLISHED=
(base) ✘-1 ~/src/deb-get [official L|…7⚑ 11] 
04:52 $ cog -r README.md 
Cogging README.md
Traceback (most recent call last):
  File "README.md", line 152, in <module>
    pretty_list = subprocess.check_output(["./deb-get", "prettylist"], encoding="utf-8")
  File "/home/phil/anaconda3/lib/python3.9/subprocess.py", line 424, in check_output
    return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
  File "/home/phil/anaconda3/lib/python3.9/subprocess.py", line 528, in run
    raise CalledProcessError(retcode, process.args,
CalledProcessError: Command '['./deb-get', 'prettylist']' returned non-zero exit status 1.
(base) ✘-4 ~/src/deb-get [official L|…7⚑ 11] 

```

@philclifford
Copy link
Member Author

when #544 is applied this will be updated and ready.
I'll try and fake it ..

@natanjunges natanjunges merged commit 180cbc1 into wimpysworld:main Sep 25, 2022
@philclifford philclifford deleted the add-nala-squashed branch September 26, 2022 01:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add nala
2 participants