search

thecodingdoc / neep

A tool to perform high-throughput survival analysis on splice variants
MIT License
2 stars 0 forks source link

Makefile #2

Closed wrathematics closed 4 years ago

wrathematics commented 4 years ago

Hi, I'm reviewing your submission for JOSS. This isn't strictly necessary, but would you consider using a Makefile? Here's a sample I used which you may freely take if you like:

CXX = g++
CXXFLAGS = -O3 -std=c++11
OMPFLAGS = -fopenmp
WARNFLAGS = -Wall -pedantic -Wextra

all: neep

SRCS=$(wildcard *.C)
OBJS=$(SRCS:.C=.o )

%.o: %.C
    $(CXX) $(CXXFLAGS) $(CPPFLAGS) $(WARNFLAGS) $(OMPFLAGS) -c $< -o $@

neep: $(OBJS)
    $(CXX) $(OBJS) -o neep $(OMPFLAGS)

clean:
    rm -rf *.o neep
scwest commented 4 years ago

I'll make one and have it committed by tomorrow.

Sean West GitHub: scwest

On Mon, Jan 27, 2020 at 2:11 PM Drew Schmidt notifications@github.com wrote:

Hi, I'm reviewing your submission for JOSS. This isn't strictly necessary, but would you consider using a Makefile? Here's a sample I used which you may freely take if you like:

CXX = g++CXXFLAGS = -O3 -std=c++11OMPFLAGS = -fopenmpWARNFLAGS = -Wall -pedantic -Wextra

all: neep SRCS=$(wildcard .C)OBJS=$(SRCS:.C=.o ) %.o: %.C $(CXX) $(CXXFLAGS) $(CPPFLAGS) $(WARNFLAGS) $(OMPFLAGS) -c $< -o $@ neep: $(OBJS) $(CXX) $(OBJS) -o neep $(OMPFLAGS) clean: rm -rf .o neep

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_thecodingdoc_neep_issues_2-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DAGAU556MYQ4LRN63OS4XHJLQ745WJA5CNFSM4KMHMHCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IJA5DZA&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=u6KSyX1ix4-0aQ2-Xzd9a_xhs3muWm7qOpqtGf4LIc8&m=FNsOnE2PiqXwFqf6EH1m3N0P0xUvGfnHo-3cSkE-s0o&s=3iN3Pq7HaNQW7CI5r_CfvDJs0QB3Zc-Hxb7qoNI9Z4M&e=, or unsubscribe https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AGAU5555JPPLGZHRPGU5MEDQ745WJANCNFSM4KMHMHCA&d=DwMCaQ&c=Cu5g146wZdoqVuKpTNsYHeFX_rg6kWhlkLF8Eft-wwo&r=u6KSyX1ix4-0aQ2-Xzd9a_xhs3muWm7qOpqtGf4LIc8&m=FNsOnE2PiqXwFqf6EH1m3N0P0xUvGfnHo-3cSkE-s0o&s=Ay2rzFwaniRIDxIwezd3S7WttTCnuOR4OlhhXfKy39c&e= .

scwest commented 4 years ago

The Makefile has been added. Thank you for your suggestion and also the filled template for writing the Makefile. I used it with a comment within the Makefile indicating that it came from you.