Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source hector, sip and pyqt from cms-externals. #1438

Merged
1 commit merged into from Mar 11, 2015
Merged

Source hector, sip and pyqt from cms-externals. #1438

1 commit merged into from Mar 11, 2015

Conversation

ghost
Copy link

@ghost ghost commented Mar 10, 2015

Hector: cms-externals/hector@365361c
Sip: cms-externals/sip@51f6c53
PyQt: cms-externals/pyqt@f909ea8

Cleanup of patch files now included in cms-externals.
+1
Tested compilation up to toolfile of each package.

%patch1 -p1
%patch2 -p1
%setup -q -n %{n}.%{realversion}
mkdir -p obj
Copy link
Contributor

Choose a reason for hiding this comment

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

For the record:

mkdir -p obj lib

should be preferred.

@ktf
Copy link
Contributor

ktf commented Mar 10, 2015

Thanks. Can you clean up at least the second issue above (well, I guess the first one as well while you are at it)?

@ghost
Copy link
Author

ghost commented Mar 11, 2015

@ktf I updated the PR. All should be fine now.

ghost pushed a commit that referenced this pull request Mar 11, 2015
Source hector, sip and pyqt from cms-externals.
@ghost ghost merged commit e56acb0 into cms-sw:IB/CMSSW_7_5_X/stable Mar 11, 2015
@ghost ghost deleted the hector_sip_pyqt_ext_75X branch March 11, 2015 09:39
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants