Reviews for lemonade
All reviews for this package from team members (across all versions).
| Reviewer | Version | Allocated | Started | Status | Completed | Comment | |
|---|---|---|---|---|---|---|---|
| siretart | 9.3.2-2 | 19 days ago | 19 days ago | rejected | 19 days ago | Sorry for the many repeated rejects. This is a hard package to review and get right. Thanks for your hard work and attention to detail on this package! Unfortunately, I found some additional issues: # `docs/assets/logo_512.png` and `docs/favicon.ico` are missing from `debian/copyright` The `debian/copyright` file lists the following files under the Microsoft/Expat stanza: ``` Files: docs/assets/favicon.ico docs/assets/logo.png src/cpp/resources/static/favicon.ico ``` Inspecting `docs/assets/logo.png` and `docs/assets/logo_512.png` visually confirms they are the same Microsoft Fluent Emoji lemon image at different resolutions (128×128 and 512×512 respectively). `NOTICE.md` line 37 explicitly confirms: *"TurnkeyML uses the Microsoft lemon emoji as an icon for the lemonade tool."* Since `logo.png` is already attributed to Microsoft Corporation under the Expat license, `logo_512.png` must receive the same attribution — it is the same artwork. It is currently covered only by the catch-all AMD/Apache-2.0 stanza, which is a misrepresentation. `docs/favicon.ico` is likewise absent from the Microsoft/Expat stanza (the stanza lists `docs/assets/favicon.ico` but not `docs/favicon.ico`, which is a distinct file). Given the consistent use of the Fluent Emoji lemon as the project icon, this file almost certainly has the same provenance. Add both `docs/assets/logo_512.png` and `docs/favicon.ico` to the existing Microsoft/Expat stanza. # `docs/assets/logo_512.png` and `docs/favicon.ico` are missing from `debian/copyright` `docs/self_hosted_runners.md` contains the following footer (lines 154–159): ``` # License [Apache 2.0 License](../LICENSE) Copyright(C) 2024-2025 Advanced Micro Devices, Inc. All rights reserved. SPDX-License-Identifier: MIT ``` The file simultaneously references the Apache 2.0 license and carries an `SPDX-License-Identifier: MIT` tag. The `debian/copyright` catch-all stanza claims it is Apache-2.0. All three cannot be correct. Clarify the intend with upstream and document the responses/conversation. # Section and Priority You chose `Section: python`; This is questionable. The binary packages `lemonade-server` and `lemonade-desktop` are not Python libraries; `lemonade-server` is an `Architecture: linux-any` compiled C++ daemon, and `lemonade-desktop` is a web application. The `python` section is intended for Python packages. A more appropriate section would be `utils` or `net`. Per Debian Policy §2.4, the section must accurately reflect the package's nature. | View |
| siretart | 9.2.0-1 | 1 month, 3 days ago | 1 month, 3 days ago | rejected | 1 month, 3 days ago | Thanks for working on this package. I found two issues in `debian/copyright` that need to be addressed before we can accept it. The `Files: *` stanza includes the line `2023 Groq Inc. (portions derived from TurnkeyML/MLAgility)` in the `Copyright` field. The parenthetical remark is not a copyright statement; it is a clarification about the origin of some code. This belongs in a `Comment` field instead. The `Copyright` field should simply read `2023 Groq Inc.` to match the copyright notice given in NOTICE.md. The favicon stanza references `src/lemonade/tools/server/static/favicon.ico`, but that path does not exist in the source tree. The actual file lives at `src/cpp/resources/static/favicon.ico`. Please update the path accordingly. | View |
| siretart | 9.0.2+dfsg-1 | 1 month, 17 days ago | 1 month, 17 days ago | rejected | 1 month, 17 days ago | requested by maintainer, new verison is in preparation to address found issues | View |
| siretart | 10.0.0-1 | 1 day ago | 1 day ago | team | - | - |