Skip to content
This repository has been archived by the owner on Jan 15, 2023. It is now read-only.

App still uses locally-embedded JAR #1

Open
medavox opened this issue May 29, 2019 · 0 comments
Open

App still uses locally-embedded JAR #1

medavox opened this issue May 29, 2019 · 0 comments
Labels
bug Something isn't working

Comments

@medavox
Copy link
Collaborator

medavox commented May 29, 2019

org.javia.arity 'Arity' is an arithmetic engine for for Java, by Mihai Preda.

It's currently used in:

com.cybrosys.basic.HistoryAdapter
com.cybrosys.basic.Logic
com.cybrosys.scientific.EventListener
com.cybrosys.scientific.HistoryAdapter
com.cybrosys.scientific.Logic
com.cybrosys.unit.Calculate

(Some of these classes may be duplicates, but that's a problem for another issue)

Originally a Freshmeat project, the project was apparently moved to Google Code some time after 2008. There are no working links for this URL; apparently it was excluded from the Google Code archive.

The code was imported from google code several times, by several different people. The most notable imports are:

For now, using the embedded JAR does work. But this project needs a better solution going forward. Checking binaries into git is never a good strategy, and relying on abandoned code that can't be updated is also a very bad idea.

Proposed Solution

A Jitpack build of Hoijui's version currently succeeds; in order to guarantee code availability going forward, this version should be forked into a repo controlled by this project.

From there, we can also include the improvements done by piusvelte

medavox added a commit that referenced this issue May 29, 2019
temporary fix for #1
still need to depend on our own arity fork
@medavox medavox added the bug Something isn't working label Jun 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant