From: chrubis@suse.cz
To: Jungsoo Son <jungsoo.son@lge.com>
Cc: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] Makefile: Disable parallel build for $(CRITICAL_MAKEFILES)
Date: Wed, 5 Jun 2013 18:11:28 +0200 [thread overview]
Message-ID: <20130605161128.GA1197@rei> (raw)
In-Reply-To: <20130603130105.GA4074@rei>
Hi!
> > > I got a an idea how to fix this in portable fashion. The key is the order
> > > of filenames the make looks for.
> > >
> > > The gnu make tries GNUmakefile first, then makefile or Makefile.
> > >
> > > So if we create GNUmakefile, which would include the Makefile and adds
> > > the .NOTPARALLEL: target it should work both with both GNU make and any
> > > other POSIX conforming make out there.
> >
> > I think your idea is good.
> > But I think that the code to distinguish GNU make and other POSIX will makes
> > the Makefile to be more complex.
> > I do not wish to complicate the Makefile more than is necessary.
> > How about using '-j1' without '.NOTPARALLEL:' option as follows.
>
> Again the same problem as previously. The POSIX specification does not
> require the -j option to exist as the standard does not take parallel
> build into the account at all. And for the Open Posix part of LTP we
> need to stick to the POSIX.
>
> And looking at the POSIX spec again, the POSIX make does not even
> support including files. So the only way how to fix this is to copy the
> top level Makefile to GNUmakefile and add the line. :(
I've looked into the openposix Makefile and it allready contains non
conforming '-j1' options. I will have closer look at the build scripts
and Makefiles.
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. A cloud service to automate IT design, transition and operations
2. Dashboards that offer high-level views of enterprise services
3. A single system of record for all IT processes
http://p.sf.net/sfu/servicenow-d2d-j
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
prev parent reply other threads:[~2013-06-05 16:10 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-21 8:57 [LTP] [PATCH] Makefile: Disable parallel build for $(CRITICAL_MAKEFILES) Jungsoo Son
2013-05-21 9:51 ` Wanlong Gao
2013-05-21 10:33 ` chrubis
2013-05-30 13:03 ` chrubis
[not found] ` <006901ce5dda$30547c60$90fd7520$@lge.com>
2013-06-03 13:01 ` chrubis
2013-06-05 16:11 ` chrubis [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130605161128.GA1197@rei \
--to=chrubis@suse.cz \
--cc=jungsoo.son@lge.com \
--cc=ltp-list@lists.sourceforge.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox