From: Willem Jan Withagen <wjw@digiware.nl>
To: Matt Benjamin <mbenjamin@redhat.com>
Cc: Ali Maredia <amaredia@redhat.com>, ceph-devel@vger.kernel.org
Subject: Re: Running Test Scripts without CMake Environment Variables
Date: Tue, 19 Apr 2016 19:57:16 +0200 [thread overview]
Message-ID: <5716717C.8000900@digiware.nl> (raw)
In-Reply-To: <1402915173.14974375.1461074347967.JavaMail.zimbra@redhat.com>
On 19-4-2016 15:59, Matt Benjamin wrote:
> Hi Willem,
>
> What are the main barriers to using CMake on FreeBSD? My only data
> point about it is nfs-ganesha builds, but my understanding has been,
> it's worked for that for a few years.
Hi Matt,
To name a few:
I did not understand that Cmake was the direction of Ceph when I started
last November...
When I started porting I got a few error messages from Cmake. And given
the fact that I'm more familiar with (g)make and autobuild, I chose that
route. I know Cmake exists, and that is about it. So that is again a
tool/language to learn. Autobuild is also far from perfect, but whacking
at it is relatively easy.
It seemed to know little about Clang, if I remember correctly, but it is
like 6 months ago.
Before investing "porting"/migrating to another build tool I prefer to
have at least a working testset. So my focus is mostly on that.
So no real practical ones, just a matter of my current choices.
But perhaps I should give it a try again and see what comes of it..
--WjW
> Matt
>
> ----- Original Message -----
>> From: "Willem Jan Withagen" <wjw@digiware.nl> To: "Ali Maredia"
>> <amaredia@redhat.com>, ceph-devel@vger.kernel.org Sent: Tuesday,
>> April 19, 2016 4:07:06 AM Subject: Re: Running Test Scripts without
>> CMake Environment Variables
>>
>> On 18-4-2016 23:53, Ali Maredia wrote:
>>> Ceph community,
>>>
>>> Recently certain tests that are run in `make check` (one's that
>>> end in .sh and a handful of others) were changed to remove
>>> relative paths and hard coded directories (".libs" was littered
>>> throughout the codebase).
>>>
>>> These test scripts pass when you run `make check` because of
>>> preset environment variables in CTest and in src/Makefile.am
>>> (CEPH_BIN, CEPH_ROOT, CEPH_LIB, CEPH_BUILD_DIR, etc.) that are
>>> meant to replace the relative paths from before. However running
>>> these tests by themselves after an autotools build without
>>> setting those variables before hand will result in failures.
>>>
>>> I understand how these recent changes disrupt developers
>>> workflows and plan on issuing a fix that defines those
>>> environment variables according to which build system you build
>>> with ASAP.
>>>
>>> Finally I strongly encourage developers to transition to using
>>> CMake and CTest (run ctest -V -R {test_name} from
>>> CMAKE_BINARY_DIR), and submit fixes and other contributions.
>>
>> Hi Ali,
>>
>> I'm trying to get a port for FreeBSD working, and started from
>> make(gmake) because Cmake did not work that well when I started.
>> And as such I do always follow the regular pattern and just use
>> the makefiles as they are. Just because running Make at the top
>> level is a rather long process. So I keep a script where I run the
>> tests that are still giving me faults. And it does complete
>> different things from what make does. Because make assuems that
>> things are going to complete with success, but in my case I'm
>> actually expecting things to go wrong. :) So then I start
>> collecting cores and logfiles for post-mortum analysis.
>>
>> So I'd like to be able to run every test on its own. And even when
>> I would migrate to Cmake, I would still want to be able to do
>> that. So they even have to work if you do not use a build system,
>> it would be very nice if things are backwards compatible.
>>
>> And yes, I also want to go to Cmake, because autobuild it an odd
>> bucket of bits, but preferably not right now.
>>
>> Thanx, --WjW -- To unsubscribe from this list: send the line
>> "unsubscribe ceph-devel" in the body of a message to
>> majordomo@vger.kernel.org More majordomo info at
>> http://vger.kernel.org/majordomo-info.html
>>
>
next prev parent reply other threads:[~2016-04-19 17:57 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <75379254.14369376.1461010122306.JavaMail.zimbra@redhat.com>
2016-04-18 21:53 ` Running Test Scripts without CMake Environment Variables Ali Maredia
2016-04-19 8:07 ` Willem Jan Withagen
2016-04-19 13:59 ` Matt Benjamin
2016-04-19 17:57 ` Willem Jan Withagen [this message]
2016-04-20 4:12 ` Matt Benjamin
2016-04-20 9:51 ` Willem Jan Withagen
2016-04-21 9:11 ` Willem Jan Withagen
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=5716717C.8000900@digiware.nl \
--to=wjw@digiware.nl \
--cc=amaredia@redhat.com \
--cc=ceph-devel@vger.kernel.org \
--cc=mbenjamin@redhat.com \
/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;
as well as URLs for NNTP newsgroup(s).