irenyak1
(Irene Nyakate)
June 2, 2019, 1:22pm
1
Hello Devs, I am having a challenge upgrading javax.servlet:jsp-api library. From the central Maven repository it shows that this dependency is now specified as
<dependency>
<groupId>javax.servlet.jsp</groupId>
<artifactId>javax.servlet.jsp-api</artifactId>
<version>2.3.3</version>
</dependency>
However in our core it is specified as
<dependency>
<groupId>javax.servlet</groupId>
<artifactId>jsp-api</artifactId>
<version>2.0</version>
</dependency>
So when I am looking out for the newest version of the library in the maven central repository, I am directed to the former which is not exactly specified as the one in the Openmrs core.
My question, do we need to do some refactoring in the code to match the new format of the library?
Here is the error log .
cc @dkayiwa , @mozzy , @ruhanga
tendomart
(tendo kiiza Martyn)
June 3, 2019, 5:46am
2
@irenyak1 did you replace the format in the pom with
<dependency>
<groupId>javax.servlet.jsp</groupId>
<artifactId>javax.servlet.jsp-api</artifactId>
<version>2.3.3</version>
</dependency>
irenyak1
(Irene Nyakate)
June 3, 2019, 5:49am
3
No @tendomart I haven’t replaced it.
dkayiwa
(Daniel Kayiwa)
June 3, 2019, 6:36am
4
First question would be, if you switch to the new format, does everything compile successfully and the reference application run without any errors?
1 Like
irenyak1
(Irene Nyakate)
June 3, 2019, 6:13pm
5
Not at all @dkayiwa . when I change the main pom and the web pom to
<dependency>
<groupId>javax.servlet.jsp</groupId>
<artifactId>javax.servlet.jsp-api</artifactId>
<version>2.3.3</version>
</dependency>
I get this error
dkayiwa
(Daniel Kayiwa)
June 3, 2019, 6:49pm
6
Do you have a pull request with the changes that you have made?
irenyak1
(Irene Nyakate)
June 3, 2019, 7:25pm
7
Let me run it again and see if it builds or not then i make a PR @dkayiwa
irenyak1
(Irene Nyakate)
June 3, 2019, 8:24pm
8
It still fails to build and here is the error log
I have made a commit to the old PR here