Bug 2423906

Summary:	Review Request: ecl - Embeddable Common Lisp
Product:	[Fedora] Fedora	Reporter:	Yaakov Selkowitz <yselkowi>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	NEW ---	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	package-review
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
URL:	https://common-lisp.net/project/ecl/
Whiteboard:
Fixed In Version:		Doc Type:	---
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:		Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Yaakov Selkowitz 2025-12-19 16:10:22 UTC

Spec URL: https://yselkowitz.fedorapeople.org/ecl.spec
SRPM URL: https://yselkowitz.fedorapeople.org/ecl-24.5.10-4.fc44.src.rpm
Description: ECL (Embeddable Common Lisp) is an implementation of the Common Lisp
language as defined by the ANSI X3J13 specification.  ECL features a
bytecode compiler and interpreter, the ability to build standalone
executables and libraries, and extensions such as ASDF, sockets, and
Gray streams.
(This was retired for lack of a maintainer, but is still maintained upstream and can be used as a backend by maxima.)
Fedora Account System Username: yselkowitz

Comment 1 Fedora Review Service 2025-12-19 16:22:08 UTC

Copr build:
https://copr.fedorainfracloud.org/coprs/build/9936115
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2423906-ecl/fedora-rawhide-x86_64/09936115-ecl/fedora-review/review.txt

Found issues:

- ecl : /usr/include/ecl/bytecodes.h ecl : /usr/include/ecl/cache.h ecl : /usr/include/ecl/config-internal.h ecl : /usr/include/ecl/config.h ecl : /usr/include/ecl/configpre-int.h ecl : /usr/include/ecl/configpre.h ecl : /usr/include/ecl/cons.h ecl : /usr/include/ecl/cs.h ecl : /usr/include/ecl/ecl-cmp.h ecl : /usr/include/ecl/ecl-inl.h ecl : /usr/include/ecl/ecl.h ecl : /usr/include/ecl/ecl_atomics.h ecl : /usr/include/ecl/external.h ecl : /usr/include/ecl/impl/math_dispatch.h ecl : /usr/include/ecl/impl/math_dispatch2.h ecl : /usr/include/ecl/impl/math_fenv.h ecl : /usr/include/ecl/impl/math_fenv_msvc.h ecl : /usr/include/ecl/internal.h ecl : /usr/include/ecl/legacy.h ecl : /usr/include/ecl/number.h ecl : /usr/include/ecl/object.h ecl : /usr/include/ecl/page.h ecl : /usr/include/ecl/stack-resize.h ecl : /usr/include/ecl/stacks.h ecl : /usr/include/ecl/threads.h ecl : /usr/share/doc/ecl/examples/android/jni/ecl_boot.h ecl : /usr/share/doc/ecl/examples/android/jni/org_lisp_ecl_EmbeddedCommonLisp.h ecl : /usr/share/doc/ecl/examples/ecl_qt/qt/hybrid_main.h 
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_devel_packages
- Unversioned so-files directly in %_libdir.
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_devel_packages
- libatomic_ops-devel is deprecated, you must not depend on it.
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/deprecating-packages/
- A package with this name already exists. Please check https://src.fedoraproject.org/rpms/ecl
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names
- Package has .a files: ecl. Illegal package name: ecl. Does not provide -static: ecl.
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/#packaging-static-libraries
- Documentation size is 5557882 bytes in 834 files. 
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_documentation

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.