Fix rpm by aashish24 · Pull Request #1705 · CDAT/cdat

aashish24 · 2015-11-24T08:40:06Z

Mostly bringing changes from release to master
Updated to odd number for master

@sankhesh @chaosphere2112 please review

…cript to link python -> python2.7

Conflicts: CMakeLists.txt

This was causing issues with RPM dependency checker Conflicts: CMakeLists.txt

RPM for 2.2.0

Conflicts: CMakeLists.txt

aashish24 · 2015-11-24T08:49:22Z

@williams13 I need to get this merged before I would like to create a release branch

durack1 · 2015-11-24T15:10:04Z

correct spelling of "PACAKGE" below - line 835

aashish24 · 2015-11-24T17:07:14Z

thanks @durack1 fixed it.

durack1 · 2015-11-24T17:13:44Z

@aashish24 on a related but separate note, how close are *.dmg UV-CDAT packages for OS X? Could these be implemented as a distribution option soon?

williams13 · 2015-11-24T17:34:57Z

Okay! Please let me know when you have created the release branch.

From: Aashish Chaudhary <notifications@github.com mailto:notifications@github.com>
Reply-To: UV-CDAT/uvcdat <reply@reply.github.com mailto:reply@reply.github.com>
Date: Tuesday, November 24, 2015 at 12:49 AM
To: UV-CDAT/uvcdat <uvcdat@noreply.github.com mailto:uvcdat@noreply.github.com>
Cc: Dean Williams <williams13@llnl.gov mailto:williams13@llnl.gov>
Subject: Re: [uvcdat] Fix rpm (#1705)

@williams13 https://github.com/williams13 I need to get this merged before I would like to create a release branch

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/1705#issuecomment-159198077.

aashish24 · 2015-11-24T17:38:14Z

@aashish24 on a related but separate note, how close are *.dmg UV-CDAT packages for OS X? Could these be implemented as a distribution option soon?

post 2.4, yes.

sankhesh · 2015-11-24T18:19:52Z

Looks good but the best validation is to generate the package and test.

aashish24 · 2015-11-24T18:53:01Z

Looks good but the best validation is to generate the package and test.

@sankhesh sounds reasonable @chaosphere2112 can you try to create an RPM using this branch?

aashish24 · 2015-11-24T19:31:19Z

@sankhesh if I don't hear back from @chaosphere2112 I may merge it since I need it for the release branch. If we find some issues then we can push it to release or master or both.

chaosphere2112 · 2015-11-24T19:33:34Z

@aashish24 I'll take a run at packaging an RPM shortly.

chaosphere2112 · 2015-11-24T21:30:20Z

@aashish24 Ran into some build issues, taking a bit longer than anticipated. Wasn't running a new enough cmake, apparently.

chaosphere2112 · 2015-11-24T21:39:07Z

... and our network is dieing horribly right now

aashish24 · 2015-11-24T23:18:57Z

@chaosphere2112 any update?

chaosphere2112 · 2015-11-24T23:20:47Z

@aashish24 Finally got my build working. I should hopefully be able to run cpack momentarily.

aashish24 · 2015-11-24T23:33:42Z

Awesome! Thanks @chaosphere2112 for the update

chaosphere2112 · 2015-11-24T23:41:09Z

Tried packaging, but wound up with these requirements:

uvcdat-1.4.0-1.x86_64 requires libQtWebKit.so.4()(64bit)
uvcdat-1.4.0-1.x86_64 requires libQtDeclarative.so.4()(64bit)
uvcdat-1.4.0-1.x86_64 requires /home/sam/work/builduvcdat/install/bin/python2.7
uvcdat-1.4.0-1.x86_64 requires /usr/local/bin/python
uvcdat-1.4.0-1.x86_64 requires /home/sam/work/builduvcdat/install/bin/python

chaosphere2112 · 2015-11-24T23:42:27Z

Ugh, looks like I was on the wrong branch. Going to repackage.

chaosphere2112 · 2015-11-24T23:59:18Z

Ran into some packaging errors, doing a clean build on the proper branch to see if that fixes it. Won't be done till after I'm out for the day (which is now), so we'll pick up on this tomorrow.

aashish24 · 2015-11-25T15:13:08Z

thanks @chaosphere2112

chaosphere2112 · 2015-11-25T16:33:45Z

Ran into the same packaging errors. Here's the log that was dumped.

chaosphere2112 · 2015-11-25T16:34:45Z

I found this bug report that mentions something similar to this issue; I'm running CMake 2.8.12, though, and they said it should have been in 2.8.8.

aashish24 · 2015-11-25T23:36:49Z

thanks @chaosphere2112. Unless we are doing something, wrong here, I would suggest we merge this branch and follow up with CMake develoepers on this issue. Sounds reasonable?

aashish24 · 2015-11-26T04:53:38Z

@chaosphere2112 I am not seeing the twice error/warning. My problem is with the freetype:

failed to create symbolic link '/home/chaudhary/tools/uvcdat/build_debug/_CPack_Packages/Linux/RPM/UVCDAT-2.3.0-Linux//home/chaudhary/tools/uvcdat/build_debug/install/Externals/include/freetype': No such file or directory

aashish24 · 2015-11-27T17:13:11Z

@chaosphere2112 both of these issues seems related but are not fault of this branch. I am merging this one and creating tag points.

Fix rpm

Sam Fries and others added 14 commits July 1, 2015 04:32

2.2 rpm initial

98771be

Made install use CMAKE_INSTALL_PATH, updated version#, added a post s…

d12875e

…cript to link python -> python2.7

Prevent any files with build tree path shebangs from being installed

36df666

Fix python path dependency and RPATH issues

a15bf6b

Conflicts: CMakeLists.txt

Fix default installation path to /usr/local/uvcdat

73b2f18

Conflicts: CMakeLists.txt

Ignore uuid while packaging

6a373a1

This was causing issues with RPM dependency checker Conflicts: CMakeLists.txt

Install different versions of UVCDAT in different directories

18b2310

Add symbolic link to python executable post install

ba264bc

Changed version to 2.2.0

de23c6a

Added build_info.txt to install

f7ffa9b

Merged

08e0e23

Removed unnecessary python fix

6f06a44

Merge pull request #1518 from UV-CDAT/package_rpm_2.2.0

8ccb7b3

RPM for 2.2.0

Merge remote-tracking branch 'origin/release' into fix_rpm

a6a5887

Conflicts: CMakeLists.txt

durack1 reviewed Nov 24, 2015
View reviewed changes

Comment thread CMakeLists.txt

Copy link
Copy Markdown

Member

durack1 Nov 24, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

correct spelling of "PACAKGE" below - line 835

Fixed typo of CMake variable

3bd8ff3

aashish24 added a commit that referenced this pull request Nov 27, 2015

Merge pull request #1705 from UV-CDAT/fix_rpm

d626305

Fix rpm

aashish24 merged commit d626305 into master Nov 27, 2015

aashish24 deleted the fix_rpm branch November 27, 2015 17:13

Conversation

aashish24 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

durack1 Nov 24, 2015

Choose a reason for hiding this comment

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

durack1 commented Nov 24, 2015

Uh oh!

williams13 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

sankhesh commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

chaosphere2112 commented Nov 24, 2015

Uh oh!

aashish24 commented Nov 25, 2015

Uh oh!

chaosphere2112 commented Nov 25, 2015

Uh oh!

chaosphere2112 commented Nov 25, 2015

Uh oh!

aashish24 commented Nov 25, 2015

Uh oh!

aashish24 commented Nov 26, 2015

Uh oh!

aashish24 commented Nov 27, 2015

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants