richb-hanover / ndt

Automatically exported from code.google.com/p/ndt
Other
1 stars 0 forks source link

Improve directory structure of the Applet #91

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Currently Applet build scripts put everything in the single directory. We 
should modify these scripts to use the standard build directories like src, bin 
and dist.

The final directory structure should look similar to the one used in janalyze.

Original issue reported on code.google.com by jslawin...@soldevelo.com on 16 Oct 2013 at 8:56

GoogleCodeExporter commented 9 years ago

Original comment by llewczyn...@soldevelo.com on 28 Oct 2013 at 9:16

GoogleCodeExporter commented 9 years ago
I submitted changes to the applet_91 branch. Can somebody review and merge?

Original comment by llewczyn...@soldevelo.com on 6 Nov 2013 at 1:31

GoogleCodeExporter commented 9 years ago
Is it still possible to build NDT as an RPM? I seem to recall maven not being 
available as an RPM which made integrating it into an RPM build environment was 
non-trivial, but I could be wrong.

Original comment by AaronMat...@gmail.com on 6 Nov 2013 at 1:34

GoogleCodeExporter commented 9 years ago
Can you share more information on how you are building ndt as an rpm? I am 
unable to find a spec file in the trunk.

Original comment by llewczyn...@soldevelo.com on 6 Nov 2013 at 2:35

GoogleCodeExporter commented 9 years ago
There used to be an issue covering this IIRC. The RPM was initially built by 
Tom at MCNC. You can grab the various files at 
http://anonsvn.internet2.edu/svn/nptoolkit/trunk/software/ndt/

Original comment by AaronMat...@gmail.com on 6 Nov 2013 at 2:42

GoogleCodeExporter commented 9 years ago
When I tried to create an RPM I noticed that files inside the Applet folder are 
not added when a user executes 'make dist'.

I added missing properties and from now there should not be a problem with 
creating an RPM.

Original comment by llewczyn...@soldevelo.com on 7 Nov 2013 at 10:07

GoogleCodeExporter commented 9 years ago
Does it require maven to build?

Original comment by AaronMat...@gmail.com on 7 Nov 2013 at 12:56

GoogleCodeExporter commented 9 years ago
yes

Original comment by llewczyn...@soldevelo.com on 7 Nov 2013 at 1:23

GoogleCodeExporter commented 9 years ago
Since Maven isn't available on RedHat systems via a convenient 'yum' install 
(i.e. it's not available in the standard RH or EPEL set of repositories), it 
means that building from source notedly harder (whether building an RPM or 
not). Is there any way to include a workaround for if the folks do not have 
maven installed?

Original comment by AaronMat...@gmail.com on 7 Nov 2013 at 1:53

GoogleCodeExporter commented 9 years ago
You would probably need to add a external repo if you want to use yum. We could 
think about embedding maven to ndt repo in the same way as we do with I2Util.

Original comment by llewczyn...@soldevelo.com on 7 Nov 2013 at 2:53

GoogleCodeExporter commented 9 years ago
What is the status of this issue? Was it merged to trunk?

Original comment by jslawin...@soldevelo.com on 15 Nov 2013 at 10:38

GoogleCodeExporter commented 9 years ago
Łukasz, can you re-implement this ticket (along with #92) without using maven? 
We have a similar directory structure in the janalyze component.

Original comment by jslawin...@soldevelo.com on 28 Nov 2013 at 8:19

GoogleCodeExporter commented 9 years ago

Original comment by skost...@soldevelo.com on 4 Feb 2014 at 9:30

GoogleCodeExporter commented 9 years ago
Reimplemented without maven usage. Changes are available on AppletIssue91 
branch. Feel free to review.

Original comment by skost...@soldevelo.com on 4 Feb 2014 at 10:11

GoogleCodeExporter commented 9 years ago
Seems fine to me. Merge to trunk at will.

Original comment by AaronMat...@gmail.com on 5 Feb 2014 at 1:34

GoogleCodeExporter commented 9 years ago
Merged into trunk

Original comment by skost...@soldevelo.com on 5 Feb 2014 at 1:47

GoogleCodeExporter commented 9 years ago

Original comment by skost...@soldevelo.com on 23 Jun 2014 at 5:55