Closed tburch closed 8 years ago
Hi Tristan -
Thanks very much for the pull request. I just wanted to touch base and let you know we received it, but we probably won't get a chance to respond to you until either the end of the week or early next week. Sorry for the delay, but we have a few urgent tasks we are finishing up and we want to get a chance to test this out properly.
-Steve
On Mon, Jun 15, 2015 at 10:15 AM, Tristan Burch notifications@github.com wrote:
I did a basic maven conversion of the project so that dependencies can be expressed. Deploying to Maven central (#3 https://github.com/scalyr/scalyr-logback/issues/3) should be fairly simple. The only todo is to set the version for scalyr-api in the parent pom.xml (it also needs to be uploaded to maven central):
<dependency> <groupId>com.scalyr</groupId> <artifactId>scalyr-api</artifactId> <version><!-- whatever version exists in maven central --></version> </dependency>
You can view, comment on, or merge this pull request online at:
https://github.com/scalyr/scalyr-logback/pull/4 Commit Summary
- Mavenize the project into two sub modules for their respective jars
File Changes
- D ant_dist/scalyrLog4J.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-0 (0)
- D ant_dist/scalyrLogback.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-1 (0)
- D build.xml https://github.com/scalyr/scalyr-logback/pull/4/files#diff-2 (55)
- D lib/log4j-1.2.17.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-3 (0)
- D lib/logback-access-1.1.2-sources.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-4 (0)
- D lib/logback-access-1.1.2.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-5 (0)
- D lib/logback-classic-1.1.2-sources.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-6 (0)
- D lib/logback-classic-1.1.2.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-7 (0)
- D lib/logback-core-1.1.2-sources.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-8 (0)
- D lib/logback-core-1.1.2.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-9 (0)
- D lib/scalyrApi.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-10 (0)
- D lib/slf4j-api-1.7.7.jar https://github.com/scalyr/scalyr-logback/pull/4/files#diff-11 (0)
- A pom.xml https://github.com/scalyr/scalyr-logback/pull/4/files#diff-12 (46)
- M readme.md https://github.com/scalyr/scalyr-logback/pull/4/files#diff-13 (27)
- A scalyr-log4j/pom.xml https://github.com/scalyr/scalyr-logback/pull/4/files#diff-14 (36)
- A scalyr-log4j/src/main/java/com/scalyr/log4j/ScalyrAppender.java https://github.com/scalyr/scalyr-logback/pull/4/files#diff-15 (81)
- R scalyr-log4j/src/test/resources/log4j.properties https://github.com/scalyr/scalyr-logback/pull/4/files#diff-16 (0)
- A scalyr-logback/pom.xml https://github.com/scalyr/scalyr-logback/pull/4/files#diff-17 (50)
- A scalyr-logback/src/main/java/com/scalyr/logback/ScalyrAppender.java https://github.com/scalyr/scalyr-logback/pull/4/files#diff-18 (89)
- R scalyr-logback/src/test/java/com/scalyr/logback/Test.java https://github.com/scalyr/scalyr-logback/pull/4/files#diff-19 (7)
- R scalyr-logback/src/test/resources/logback.groovy https://github.com/scalyr/scalyr-logback/pull/4/files#diff-20 (8)
- D src/com/scalyr/log4j/ScalyrAppender.java https://github.com/scalyr/scalyr-logback/pull/4/files#diff-21 (80)
- D src/com/scalyr/logback/ScalyrAppender.java https://github.com/scalyr/scalyr-logback/pull/4/files#diff-22 (87)
Patch Links:
- https://github.com/scalyr/scalyr-logback/pull/4.patch
- https://github.com/scalyr/scalyr-logback/pull/4.diff
— Reply to this email directly or view it on GitHub https://github.com/scalyr/scalyr-logback/pull/4.
I did a basic maven conversion of the project so that dependencies can be expressed. Deploying to Maven central (#3) should be fairly simple. The only todo is to set the version for
scalyr-api
in the parentpom.xml
(it also needs to be uploaded to maven central):